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

[bug] isReleaseBranch check fails for PR to release branches #338

Closed
1 of 2 tasks
lukekarrys opened this issue Jul 26, 2023 · 2 comments · Fixed by #360
Closed
1 of 2 tasks

[bug] isReleaseBranch check fails for PR to release branches #338

lukekarrys opened this issue Jul 26, 2023 · 2 comments · Fixed by #360

Comments

@lukekarrys
Copy link
Contributor

lukekarrys commented Jul 26, 2023

a release branch is checked based on matching the pattern release/v*. making a PR to a release branch wont match that pattern so that check will be false which will reset some templated files to the incorrect state.

Possible solutions:

  • make isReleaseBranch config only that needs to be set manually per branch
  • make branch pattern match PR branches and require PRs to release branches be named according to that pattern (eg release/v9-update-config)
@rotu
Copy link
Contributor

rotu commented Sep 14, 2023

Is this issue responsible for:

​FAIL ​ test/check/dogfood.js
✖ should be equivalent strictly
...
  +  Object {
  +    "title": "The module file package.json needs to be updated:",
  +    "body": Array [
  +      String(
  +        workspace\\test-workspace\\package.json
  +        ========================================
  +        "repository.url" is "https://github.com/npm/template-oss.git", expected
"https://github.com/rotu/npm-template-oss.git"
  +      ),
  +    ],
  +    "solution": "npx template-oss-apply --force",
  +  },

@lukekarrys
Copy link
Contributor Author

@rotu that is due to the autodetection of the origin by git. If you've forked the repo make sure that npm/template-oss is the origin and the remote url for your fork is named something different (eg fork):

❯ git remotes
fork	git@github.com:lukekarrys/template-oss.git (fetch)
fork	git@github.com:lukekarrys/template-oss.git (push)
origin	git@github.com:npm/template-oss.git (fetch)
origin	git@github.com:npm/template-oss.git (push)

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 a pull request may close this issue.

2 participants