Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add link to ASF privacy policy #91

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

cpoerschke
Copy link

Hello. This draft PR is in response to seeing the "Apache Project Website Checks" here: https://whimsy.apache.org/site/project/opennlp -- I'm not familiar with the .ad syntax or what wording the project might wish to use etc. -- will check the 'allow edits by maintainers' button i.e. edits and revisions are welcome. Thank you.

@kinow
Copy link
Member

kinow commented Jan 22, 2025

(+1 to add it, but someone needs to check the syntax -- thanks @cpoerschke !)

@mawiesne mawiesne requested a review from rzo1 January 22, 2025 11:19
@mawiesne
Copy link
Contributor

mawiesne commented Jan 22, 2025

@cpoerschke syntax example below:

https://github.com/apache/opennlp[Github opennlp,window=_blank]

Just the URL combined with a label in square brackets for the link text, followed by a parameter for a separate browser window / tab.

@mawiesne mawiesne marked this pull request as draft January 22, 2025 11:36
@mawiesne mawiesne requested a review from jzonthemtn January 22, 2025 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants