Skip to content

Update links in neuro-rse.md #1716

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 9, 2025
Merged

Update links in neuro-rse.md #1716

merged 1 commit into from
May 9, 2025

Conversation

rly
Copy link
Contributor

@rly rly commented May 8, 2025

Description

  • The NWB website links have changed as we, the NWB team, transition to using a new website. We are working on getting the redirects right, but in the meantime, I fixed some links.
  • I also updated links to events for specific years to the latest event

Checklist:

  • I have previewed changes locally or with CircleCI (runs when PR is created)
  • I have completed any content reviews, such as getting input from relevant working groups. If no, please note this and wait to post the PR to the #website channel until the content has been settled.

When you are ready for a technical review/merge, post the for the link for the PR in the US-RSE Slack (#website) to ask for reviewers.

@crd477
Copy link
Contributor

crd477 commented May 8, 2025

Preview here.

crd477
crd477 approved these changes May 8, 2025
View reviewed changes
Copy link
Contributor

@crd477 crd477 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@crd477
Copy link
Contributor

crd477 commented May 9, 2025 •
edited
Loading

The failing links work for me and are unrelated to this PR anyway. Is this OK to merge or do you want someone else from the AG to check it?

@rly
Copy link
Contributor Author

rly commented May 9, 2025

This is OK to merge now. Thanks!

@crd477 crd477 merged commit e69aeff into USRSE:main May 9, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@crd477 crd477 crd477 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rly @crd477