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

docs: automatic updating using github actions #1456

Merged
merged 7 commits into from
Jun 18, 2024

Conversation

pavelzw
Copy link
Contributor

@pavelzw pavelzw commented May 29, 2024

depends on #1446

i'm still experimenting a bit with the output format, though, so the cli interface of pixi-diff-to-markdown is not super stable yet. should we include an updater in this repository as well?

@pavelzw pavelzw force-pushed the docs-lockfile-autoupdate branch from ab84702 to bbc482e Compare May 29, 2024 07:53
@pavelzw pavelzw force-pushed the docs-lockfile-autoupdate branch from bbc482e to 1342ea0 Compare May 29, 2024 07:53
@ruben-arts
Copy link
Contributor

Amazing @pavelzw,

I think adding it here would be great, give me that dogfood!

I'm moving it into draft untill its ready, to keep a clear overview.

@ruben-arts ruben-arts marked this pull request as draft May 30, 2024 05:40
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this currently results in the CI not being triggered when the PR is opened: https://github.com/peter-evans/create-pull-request/blob/main/docs/concepts-guidelines.md#triggering-further-workflow-runs

a workaround for this is to just close and re-open the PR.
otherwise we would need to add PATs, a GitHub App or an SSH deploy key for this to work

I think the close and reopen method is the easiest for this repo, especially when the PR is only coming around once a month

@pavelzw
Copy link
Contributor Author

pavelzw commented Jun 16, 2024

@pavelzw pavelzw marked this pull request as ready for review June 16, 2024 23:23
Copy link
Contributor Author

Choose a reason for hiding this comment

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

you need to do this in your repo as well

@tdejager tdejager merged commit 4653639 into prefix-dev:main Jun 18, 2024
27 checks passed
@pavelzw pavelzw deleted the docs-lockfile-autoupdate branch June 18, 2024 12:50
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