Skip to content

Added release PR action#48

Merged
andrewnester merged 5 commits intomainfrom
release-action
Oct 2, 2023
Merged

Added release PR action#48
andrewnester merged 5 commits intomainfrom
release-action

Conversation

@andrewnester
Copy link
Copy Markdown
Contributor

@andrewnester andrewnester commented Sep 29, 2023

Manually triggered with act, here's the result PR #50

Act command

act -j release-pr --input version=0.206.1-dev -s GITHUB_TOKEN="$(gh auth token)"

@andrewnester andrewnester force-pushed the release-action branch 2 times, most recently from 59b5e14 to ebbc4ed Compare September 29, 2023 15:25
@andrewnester andrewnester changed the title [WIP] Added release PR action Added release PR action Sep 29, 2023
Copy link
Copy Markdown
Collaborator

@pietern pietern left a comment

Choose a reason for hiding this comment

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

We still need to manually trigger this at the moment. How do you want to integrate it?

- uses: actions/checkout@v3
with:
fetch-depth: 0
fetch-tags: true
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

No need for full checkout if we're making a change on top.

@andrewnester
Copy link
Copy Markdown
Contributor Author

@pietern as we discussed before we'll leverage webhooks on CLI repo

@andrewnester andrewnester merged commit 3fbabb8 into main Oct 2, 2023
@andrewnester andrewnester deleted the release-action branch October 2, 2023 10:12
ilia-db added a commit to databricks/databricks-vscode that referenced this pull request Dec 21, 2023
It will be triggered automatically after we add a dispatch logic to the
CLI repo ([in the release
action](https://github.com/databricks/cli/blob/main/.github/workflows/release.yml#L36)).

The approach is the same as we've introduced for automatic PRs to the
setup-cli repo: databricks/setup-cli#48
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.

2 participants