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

New package: PSSFSS v0.1.1 #36694

Merged
merged 1 commit into from
May 17, 2021
Merged

Conversation

JuliaRegistrator
Copy link
Contributor

UUID: 6b20a5d4-3c6c-44cd-883b-1480592d72be
Repo: https://github.com/simonp0420/PSSFSS.jl.git
Tree: 4179aa3a2782d92bec2bd5c04d186209c3a797fc

Registrator tree SHA: c7e033175c3b9b466fb2cc8beab47042878a66b0
@github-actions
Copy link
Contributor

Your new package pull request does not meet the guidelines for auto-merging. Please make sure that you have read the General registry README. The following guidelines were not met:

  • Name does not meet all of the following: starts with an uppercase letter, ASCII alphanumerics only, not all letters are uppercase.

Note that the guidelines are only required for the pull request to be merged automatically. However, it is strongly recommended to follow them, since otherwise the pull request needs to be manually reviewed and merged by a human.

After you have fixed the AutoMerge issues, simple retrigger Registrator, which will automatically update this pull request. You do not need to change the version number in your Project.toml file (unless of course the AutoMerge issue is that you skipped a version number, in which case you should change the version number).

If you do not want to fix the AutoMerge issues, please post a comment explaining why you would like this pull request to be manually merged.

Since you are registering a new package, please make sure that you have also read the package naming guidelines: https://julialang.github.io/Pkg.jl/dev/creating-packages/#Package-naming-guidelines-1


If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.

@simonp0420
Copy link

simonp0420 commented May 13, 2021

[noblock]
Automerging failed because the name contains all uppercase letters.
I would like to request an exception to the naming guidelines because the name is an acronym for Polarization Selective Surfaces/Frequency Selective Surfaces. It is aimed at a narrow audience of antenna and electromagnetic engineers who are familiar with these terms and acronyms, and I don't see a natural way to insert lower case letters in this name. Any name that would be understandable to a wider audience would have to be extremely long.

@giordano giordano merged commit 2f37173 into master May 17, 2021
@giordano giordano deleted the registrator/pssfss/6b20a5d4/v0.1.1 branch May 17, 2021 16:08
@simonp0420
Copy link

Thanks!

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.

None yet

3 participants