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

Only apply milestone to PR if it exists [DOC-268] #16

Merged
merged 2 commits into from
Nov 22, 2024

Conversation

JackPGreen
Copy link
Contributor

@JackPGreen JackPGreen commented Nov 22, 2024

If a milestone doesn't exist, an error is thrown when doing a backport.

Not all repositories use milestones - instead it should only be applied to the PR, if it exists.

Fixes: DOC-268

If a milestone doesn't exist, [an error is thrown when doing a backport](https://github.com/JackPGreen2/hz-docs/actions/runs/11963090837/job/33352851231#step:7:96).

Not all repositories use milestones - instead it should only be applied to the PR, _if it exists_.

Fixes: [DOC-268](https://hazelcast.atlassian.net/browse/DOC-268)
@JackPGreen JackPGreen merged commit a71029f into hazelcast:master Nov 22, 2024
3 checks passed
JackPGreen added a commit to hazelcast/hz-docs that referenced this pull request Dec 2, 2024
…8] (#1396)

In [DOC-255](https://hazelcast.atlassian.net/browse/DOC-255), branch
protection was added - preventing direct `push`es.

Instead, the backport workflow should create PRs to backport the
changes. This gives the _possibility_ for pre-merge checks to be run
(i.e. preventing introducing dead links etc).

Leverages [_existing_ `backport`
tool](https://github.com/hazelcast/backport), but exposed some issues
which need to be addressed **before this can be merged**:
- hazelcast/backport#14
- hazelcast/backport#15
- hazelcast/backport#16
- hazelcast/backport#17

Tested in a [test repo](https://github.com/JackPGreen2/hz-docs), where a
[dummy PR](JackPGreen2#5) was
[successfully
backported](JackPGreen2#16). However,
due to the complexity of fork-triggering, it wil need to be re-tested
once merged.

Fixes: [DOC-268](https://hazelcast.atlassian.net/browse/DOC-268)

[DOC-255]:
https://hazelcast.atlassian.net/browse/DOC-255?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
[DOC-268]:
https://hazelcast.atlassian.net/browse/DOC-268?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
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