Skip to content

fix github url #39

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

Merged
merged 1 commit into from
Jun 19, 2018
Merged

fix github url #39

merged 1 commit into from
Jun 19, 2018

Conversation

MasseGuillaume
Copy link
Contributor

@MasseGuillaume MasseGuillaume commented Jun 12, 2018

this is a bug introduced in:
d63d4ea#diff-04c6e90faac2675aa89e2176d2eec7d8R49

per
https://github.com/scalacenter/scalafix/blob/8490db2edf3ef9aa6291870c04e56fa37ce4537d/scalafix-reflect/src/main/scala/scalafix/internal/reflect/GitHubUrlRule.scala#L33

the url format is github:org/repo/version

we use NewCollections as the version, but it get's lowercased

scalafix github:scala/scala-collection-compat/NewCollections

gives me

error: 404 - not found https://raw.githubusercontent.com/scala/scala-collection-compat/master/scalafix/rules/src/main/scala/fix/Scalacollectioncompat_newcollections.scala

workaround

scalafix https://raw.githubusercontent.com/scala/scala-collection-compat/master/scalafix/rules/src/main/scala/fix/Scalacollectioncompat_NewCollections.scala

@julienrf
Copy link
Contributor

Can you resolve the conflicts please?

@MasseGuillaume
Copy link
Contributor Author

@julienrf done, ready to merge.

@julienrf julienrf merged commit 3688ecf into scala:master Jun 19, 2018
@MasseGuillaume MasseGuillaume deleted the fix-github-url branch June 19, 2018 09:49
martijnhoekstra pushed a commit to martijnhoekstra/scala-collection-compat that referenced this pull request Nov 9, 2022
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