Skip to content

Conversation

@giordano
Copy link
Member

This check shows this message:

Also, note the warning: This looks like a new registration that registers version 0.1.1.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call
register() again after making the fix. This will update the Pull request.

This is confusing (it prompts users to change version number even if they do not
have to) and false: it is not true that changing the version number and
registering again will update the pull request, on the contrary it will open a
new pull request which will never be merged because it will conflict with the
previously opened PR.

This check shows this message:

```
Also, note the warning: This looks like a new registration that registers version 0.1.1.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call
register() again after making the fix. This will update the Pull request.
```

This is confusing (it prompts users to change version number even if they do not
have to) and false: it is not true that changing the version number and
registering again will update the pull request, on the contrary it will open a
new pull request which will never be merged because it will conflict with the
previously opened PR.
@nkottary nkottary closed this May 12, 2022
@nkottary nkottary reopened this May 12, 2022
status = Symbol[:new_package, :new_package_label,
:not_standard_first_version],
status = Symbol[:new_package, :new_package_label],
regbranch = (error = false, warning = true,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Also needs to set warning = false.

status = Symbol[:new_package, :new_package_label, :version_zero,
:not_standard_first_version],
status = Symbol[:new_package, :new_package_label, :version_zero],
regbranch = (error = true, warning = true,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same.

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.

4 participants