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

Add small-time #57

Merged
merged 3 commits into from
Jan 24, 2025
Merged

Add small-time #57

merged 3 commits into from
Jan 24, 2025

Conversation

carolinefrasca
Copy link
Collaborator

Add @thatstoasty's small-time, which is a dependency of lightbug_http

Checklist

  • My recipe.yaml file specifies which version(s) of MAX is compatible with my project (see here for an example). If not, my package is compatible with both 24.5 and 24.6.
  • License file is packaged (see here for an example).
  • Set the build number to 0 (for new packages, or if the version changed).
  • Bumped the build number (if the version is unchanged).

@carolinefrasca
Copy link
Collaborator Author

@thatstoasty any issues with us adding this? It's used by @saviorand's lightbug_http and we're running into issues with the CI due to small-time being on the other Mojo community channel - hopefully this would resolve it

@thatstoasty
Copy link
Contributor

No problem on my end!

@carolinefrasca carolinefrasca merged commit 4f6bd59 into main Jan 24, 2025
6 checks passed
version: "0.0.1"

package:
name: "mist"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed small_time wasn't added to the channel on Prefix yet, I think the package name is wrong here @carolinefrasca ? Could that be the issue?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We finally got this sorted! Thank you for flagging!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants