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

Clarify GitHub App workflow in README #618

Merged

Conversation

SpecialThing44
Copy link
Contributor

@SpecialThing44 SpecialThing44 commented Jul 8, 2024

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.

Copy link
Member

@alejandrohdezma alejandrohdezma 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 very much @SpecialThing44! 😄 It is great! Left some minor comments or corrections.

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated
Comment on lines 154 to 159
## 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).

Copy link
Member

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.

Suggested change
## 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).

Copy link
Contributor Author

@SpecialThing44 SpecialThing44 Jul 9, 2024

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.

@SpecialThing44 SpecialThing44 force-pushed the document-github-token branch 2 times, most recently from b7173b9 to 118daa4 Compare July 9, 2024 13:43
README.md Outdated Show resolved Hide resolved
- Remove unclear/inaccurate documentation
- Provide more complete example for using the default GitHub Actions App
@alejandrohdezma alejandrohdezma merged commit 3ba3c62 into scala-steward-org:master Jul 10, 2024
2 checks passed
Copy link
Contributor

Code Coverage

Package Line Rate Branch Rate Complexity Health
core 100% 100% 0
modules 69% 88% 0
Summary 70% (475 / 677) 89% (73 / 82) 0

Copy link
Contributor

A snapshot release has been created as snapshots/618.

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.

github-actions bot added a commit that referenced this pull request Jul 10, 2024
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.

Document that an application is not required
2 participants