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 the readme to reflect the latest version #66

Merged
merged 2 commits into from Jan 17, 2023
Merged

Conversation

devoncarew
Copy link
Member

  • update the readme to reflect the latest version

@devoncarew
Copy link
Member Author

cc @isoos for review

I don't know why this repo doesn't use a traditional changelog.md file; perhaps for visibility on the github action's marketplace page? I don't know whats typical for actions -

README.md Outdated
@@ -54,7 +54,7 @@ jobs:

steps:
- uses: actions/checkout@v3
- uses: dart-lang/setup-dart@v1.3
- uses: dart-lang/setup-dart@v1.4
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we can use @v1? /cc @mit-mit

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, that's what I would suggest users too.

That way they get the latest logic...

Copy link
Contributor

@isoos isoos left a comment

Choose a reason for hiding this comment

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

lgtm

@devoncarew devoncarew merged commit 4cbd7af into main Jan 17, 2023
@devoncarew devoncarew deleted the update_readme branch March 1, 2023 07:22
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