-
Notifications
You must be signed in to change notification settings - Fork 129
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
Repository rename check is case sensitive #901
Comments
@jedwards1211 fyi |
@kc1r74p Okay sorry for the trouble, at first I thought @gr2m @babblebey I think we should add an option to disable this check...it seems hard to exhaustively determine what all the possible cases in the wild would be. |
@jedwards1211 actually thanks for the hint, looking at it again the org is in the Besides that the string should be equal. |
I agree @jedwards1211 that it can be very exhaustive to determine possible case 😭 But, add option to skip a check in the |
@babblebey I mean to disable the repo rename check only, not all of the other verify checks. |
@kc1r74p okay, I didn't think about case sensitivity, btw are you using a private GitHub enterprise deployment? |
Yes, we are using GHES 3.11.13. |
I'd avoid adding new options if at all possible |
🎉 This issue has been resolved in version 10.1.7 🎉 The release is available on: Your semantic-release bot 📦🚀 |
If anyone with permission could update the title to indicate the issue was about case sensitivity, it might help other people who run into this see that it's been fixed. Thanks for bringing this to my attention @kc1r74p! |
done! Happy to rename it to something else for more clarity, let me know what you'd prefer |
Thanks, I think it's good now |
Since the introduction of #886 in v10.1.5 the
repositoryUrl
vsclone_url
check seems to have false positives.We use the release as part of a monorepo, so it probably fetches the repo URL from the .git as the multiple package.json do not have the repo url in it in our case.
Details below, just adjusted host + repo. The
https://[secure]@
part is 1:1 from the log and probably the reason why it detects the false positive.The plugin loading does not seem to allow to enforce an older version ?
Had to manually adjust the packages after install to workaround the issue for now.
Would be nice to ignore the user section before the host in some fix.
Thanks
The text was updated successfully, but these errors were encountered: