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

pySCG: main readme draft note links #808

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

myteron
Copy link
Contributor

@myteron myteron commented Feb 27, 2025

Adding links to the draft note to allow finding:

  • the web version when on GitHub
  • the GitHub version when viewing the web version

@myteron
Copy link
Contributor Author

myteron commented Mar 6, 2025

Hi @david-a-wheeler since we hope to have our own ossf repo at some stage, how persistent is https://best.openssf.org/Secure-Coding-Guide-for-Python/ as a URL?

@david-a-wheeler
Copy link
Contributor

Hi @david-a-wheeler since we hope to have our own ossf repo at some stage, how persistent is https://best.openssf.org/Secure-Coding-Guide-for-Python/ as a URL?

The "best.openssf.org" domain is owned by openssf (obviously) & controlled by the Best Practices WG. So it should be quite stable.

If you create your own repo, it's probably better to create its own domain ("pythonguide.openssf.org" maybe?) instead of trying to merge things. Then you'd set up a redirect from the current URL so people are redirected to the new location. The usual way to do this is by creating an HTML file and using the <meta> tag, like this:

<meta http-equiv="REFRESH" content="0;url=https://pythonguide.openssf.org">

@david-a-wheeler
Copy link
Contributor

It's possible have a separate repo, and then merge the results, but then you have to create extra steps to merge the results, extra steps to trigger things, etc. If there's no need to make things complicated, I suggest keeping things simple.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants