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

trivial fix to make the tributors badge go to the correct section #1190

Merged
merged 1 commit into from
Jun 21, 2023

Conversation

crd477
Copy link
Contributor

@crd477 crd477 commented Jun 20, 2023

Description

This is just a trivial fix to make the link on the tributors badge go to the correct section.

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.

Signed-off-by: Chad Dougherty <crd@acm.org>
@kescobo kescobo merged commit f257bea into USRSE:main Jun 21, 2023
2 checks passed
@crd477 crd477 deleted the fix/tributors-badge branch June 21, 2023 19:10
@vsoch
Copy link
Member

vsoch commented Jun 21, 2023

Here is the issue you want: all-contributors/all-contributors#615. For context, having (or not having) an emoji dictates that dash being correct or not. At one point (in the issue) it was a bug. A quick fix would be to add an emoji.

@vsoch
Copy link
Member

vsoch commented Jun 21, 2023

Try adding the badge template explicitly, e.g., an example https://github.com/ruby-gh-actions/sandbox/blob/main/.all-contributorsrc

@crd477
Copy link
Contributor Author

crd477 commented Jun 21, 2023

Thanks. The badge template is probably the best solution.

crd477 added a commit to crd477/usrse.github.io that referenced this pull request Jun 21, 2023
Contributors section

This is the more complete fix for USRSE#1190

Signed-off-by: Chad Dougherty <crd@acm.org>
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.

None yet

3 participants