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

Allow specifying packages with . in their names #140

Merged
merged 2 commits into from
Feb 12, 2024

Conversation

ayys
Copy link
Member

@ayys ayys commented Feb 12, 2024

this is done by internally replacing all the . with -.

@ayys ayys force-pushed the allow-period-in-package-name branch from b09159f to c22e03a Compare February 12, 2024 08:48
this is done by internally replacing all the `.` with `-`.
@ayys ayys force-pushed the allow-period-in-package-name branch 6 times, most recently from 3eac02f to e5535e2 Compare February 12, 2024 09:20
Copy link

github-actions bot commented Feb 12, 2024

⏱ Workflow Timer ⏱

Make sure you keep an eye on build times!

One of this project's goals is to keep CI runs under 5 minutes so developers can maintain fast edit-compile-test cycles.

Run Compile and Test
master 2mins 51s
bafa228 2mins 36s

🤖 Beep. Boop. I'm a bot. If you find any issues, please report them to https://github.com/Michael-F-Bryan/workflow-timer.

Sometimes the rust toolchain was not properly installed

this installs the rust toolchain before running cargo. Fixes errors about rustc
being out of date.
@ayys ayys force-pushed the allow-period-in-package-name branch from e5535e2 to bafa228 Compare February 12, 2024 09:31
@ayys ayys merged commit 0b53563 into master Feb 12, 2024
5 checks passed
@ayys ayys deleted the allow-period-in-package-name branch February 12, 2024 09:41
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.

1 participant