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

Update UUID module for Chapel v2.0 #71

Merged
merged 7 commits into from
Dec 11, 2024
Merged

Conversation

astatide
Copy link
Contributor

This PR adds a new version of the UUID module, which has been updated to work properly with Chapel 2.0. It also updates to the new repo location on Codeberg.

@arezaii
Copy link
Collaborator

arezaii commented Apr 29, 2024

It looks like we (mason) need to update how we validate license names since SPDX archived their repo and changed where they publish the list.

Bricks/UUID/1.0.3.toml Outdated Show resolved Hide resolved
@arezaii
Copy link
Collaborator

arezaii commented May 23, 2024

I recently merged chapel-lang/chapel#25062, which should fix the issue with the license names once you run mason publish --refresh-licenses

@astatide
Copy link
Contributor Author

Aha, yay, looks like it works now. Thank you!

@lydia-duncan lydia-duncan requested a review from arezaii June 28, 2024 17:08
@astatide
Copy link
Contributor Author

Oh, I forgot this was still open; I can't actually merge it on my own. It looks like everything is ready to go, though, so is there anything else required for merging?

@lydia-duncan
Copy link
Member

Ahmad's pretty busy with release stuff at the moment, but I'll set a reminder to either ask him and merge it myself or have him merge it when we're past the release (though all of that may be impacted by upcoming end of year vacation time)

@lydia-duncan
Copy link
Member

Thanks for reminding us, though, we really appreciate it :)

@astatide
Copy link
Contributor Author

No problem and no rush! I was setting up a project with mason and testing how to actually add dependencies, then realized I pulled in the old version and couldn't remember if this is the module I had updated or not!

@arezaii arezaii merged commit 2831a92 into chapel-lang:master Dec 11, 2024
1 check passed
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.

3 participants