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

Article on best practices in multi-project CI/CD #2249

Merged

Conversation

ryanmrichard
Copy link
Contributor

This PR is the article mentioned in #2248.

@github-actions github-actions bot added the external contribution Issue or PR from outside of the BSSw editorial team label Feb 7, 2025
@bernhold
Copy link
Member

@ryanmrichard, you seem to be using a number of tinyurl links in the article. Is there a particular reason for that? I would generally prefer to expose the full direct link instead of a shortened reference. It allows readers to use the hover preview of the link to get useful information and decide whether they want to follow it or not. It is generally considered more friendly from a cybersecurity standpoint too.

Unless there's a strong reason, I'm going to replace the tinyurls with their expansions. Please let me know ASAP. Thanks

@bernhold bernhold added content: blog For blog content preview Merge PR to the 'preview' branch to allow display on preview.bssw.io labels Feb 24, 2025
@ryanmrichard
Copy link
Contributor Author

@bernhold I used the tinyurl links as a force of habit. In my other projects I'm obsessive about not going over 80 characters per line and markdown doesn't have a great way for splitting long URLs (at least that I've found). I think your changes look good!

@bernhold bernhold merged commit ff25354 into betterscientificsoftware:main Feb 26, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content: blog For blog content external contribution Issue or PR from outside of the BSSw editorial team preview Merge PR to the 'preview' branch to allow display on preview.bssw.io
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants