Skip to content

Update README.md#7

Open
smorimoto wants to merge 2 commits into
tarides:mainfrom
smorimoto:update-readme
Open

Update README.md#7
smorimoto wants to merge 2 commits into
tarides:mainfrom
smorimoto:update-readme

Conversation

@smorimoto

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Sora Morimoto <sora@morimoto.io>

@Leonidas-from-XIV Leonidas-from-XIV left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, I have one small change I'd like to have before merging.

Comment thread README.md Outdated
Check-Changelog:
name: Check Changelog Action
runs-on: ubuntu-20.04
runs-on: ubuntu-latest

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Generally I'd prefer not to push people to setups that might break, so I'd rather have a fixed version here.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

For such a simple action it should be fine to use ubuntu-latest. Usually the jobs are independent, I don't think anyone is going add more steps after the changelog action, so they'll use their preferred OS.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes

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.

3 participants