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

Validate PackageName #377

Merged
merged 1 commit into from
Jul 24, 2024
Merged

Validate PackageName #377

merged 1 commit into from
Jul 24, 2024

Conversation

abelsiqueira
Copy link
Owner

@abelsiqueira abelsiqueira commented Jul 24, 2024

Uses regex to make sure that the package name starts with a capital letter and contains only letters and numbers.

Closes #373

Related issues

Checklist

Copy link

codecov bot commented Jul 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (20aaa32) to head (3e2e9d0).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #377   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines           79        79           
=========================================
  Hits            79        79           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Uses regex to make sure that the package name starts
with a capital letter and contains only letters and numbers.

Closes #373
@abelsiqueira
Copy link
Owner Author

@jhidding, would you like to review this?

Copy link

@jhidding jhidding left a comment

Choose a reason for hiding this comment

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

Looks solid!

@abelsiqueira abelsiqueira merged commit 5228d5b into main Jul 24, 2024
7 checks passed
@abelsiqueira abelsiqueira deleted the 373-validate-package-name branch July 24, 2024 09:34
@abelsiqueira
Copy link
Owner Author

Thanks for review!

@allcontributors please add @jhidding for review

@abelsiqueira
Copy link
Owner Author

@allcontributors please add @jhidding for review

Copy link
Contributor

@abelsiqueira

I've put up a pull request to add @thanks! 🎉

I've put up a pull request to add @jhidding! 🎉

Copy link
Contributor

@abelsiqueira

We had trouble processing your request. Please try again later.

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

Successfully merging this pull request may close these issues.

[Bug] second .jl is added when package name is entered with .jl attached
2 participants