-
Notifications
You must be signed in to change notification settings - Fork 47
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
Clarify GitHub App workflow in README #618
Clarify GitHub App workflow in README #618
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you very much @SpecialThing44! 😄 It is great! Left some minor comments or corrections.
README.md
Outdated
## Configuration | ||
|
||
There is a `repo-config` option in usage which is a path to a configuration file. It defaults to `.github/.scala-steward.conf`. | ||
|
||
See the [configuration documentation](/~https://github.com/scala-steward-org/scala-steward/blob/main/docs/repo-specific-configuration.md). | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this is redundant, as it is already specified in the Usage
section.
## Configuration | |
There is a `repo-config` option in usage which is a path to a configuration file. It defaults to `.github/.scala-steward.conf`. | |
See the [configuration documentation](/~https://github.com/scala-steward-org/scala-steward/blob/main/docs/repo-specific-configuration.md). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't mind leaving it only in the usage section, but I'm going to add the link in there then, as it is in a separate repo. See latest changes.
b7173b9
to
118daa4
Compare
- Remove unclear/inaccurate documentation - Provide more complete example for using the default GitHub Actions App
118daa4
to
1e070dd
Compare
A snapshot release has been created as You can test it out with: uses: scala-steward-org/scala-steward-action@snapshots/618 It will be automatically recreated on any change to this PR. |
To close #617
Prompted by wanting to use this action for a single repository and not wanting to create an app/manage secrets.
See testing /~https://github.com/SpecialThing44/scala-steward-action-test
Github Actions app does not have permission to write to other repos.
Some other minor clarifications that confused me when I read them.