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

Update incorrect project name in README #8

Merged
merged 1 commit into from
Dec 6, 2019
Merged

Update incorrect project name in README #8

merged 1 commit into from
Dec 6, 2019

Conversation

martinemde
Copy link
Contributor

@martinemde martinemde commented Dec 6, 2019

Bug fix

Description

Reference to stardardrb-linter-action is incorrect.

Why should this be added

I initially copied the first line in the readme a few weeks ago and gave up when it didn't work. I didn't notice the project name was wrong and just thought the action was broken. I decided to try again today and it works with the correct name.

Checklist

  • My change is so simple that I just want an extra box to check.

Reference to stardardrb-linter-action is incorrect.
Copy link
Owner

@andrewmcodes andrewmcodes left a comment

Choose a reason for hiding this comment

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

Thank you @martinemde! I have a lot of changes I need to make on this that I haven’t gotten around to yet bc I haven’t heard of anyone using this but if you are I can bump those improvements up on my todo list.

@andrewmcodes andrewmcodes merged commit a98e147 into andrewmcodes:master Dec 6, 2019
@martinemde
Copy link
Contributor Author

Nice. Does github actions tell you if anyone is using it? It's one of only a few github actions for ruby, so I'm suprised more people aren't using it.

@martinemde martinemde deleted the patch-1 branch December 9, 2019 22:40
@andrewmcodes
Copy link
Owner

@martinemde they do not unfortunately. I know my rubocop action is much more popular only bc of Issues and PR’s.

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.

2 participants