Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: add clean() test for build metadata #658

Merged
merged 1 commit into from
Nov 30, 2023
Merged

chore: add clean() test for build metadata #658

merged 1 commit into from
Nov 30, 2023

Conversation

jethrodaniel
Copy link
Contributor

Adds a test for semver.clean that demonstrates how the function strips build metadata.

References

@wraithgar
Copy link
Member

Was there more you wanted to do here?

@jethrodaniel
Copy link
Contributor Author

jethrodaniel commented Nov 30, 2023

Was there more you wanted to do here?

Nope, that's all.

Personally, I didn't realize that semver considers versions that differ by only the build metadata to be equal in precedent...

so when I noticed that semver.clean strips it in an unrelated project, the first place I came to look for clarification on the intended behavior was here, and then noticed that the tests alone couldn't confirm that for me.

I then read the semver spec more closely and everything made sense.

@wraithgar
Copy link
Member

Nope, that's all.

You can mark this as ready then.

@wraithgar wraithgar self-assigned this Nov 30, 2023
@jethrodaniel jethrodaniel marked this pull request as ready for review November 30, 2023 19:54
@jethrodaniel jethrodaniel requested a review from a team as a code owner November 30, 2023 19:54
@jethrodaniel
Copy link
Contributor Author

Nope, that's all.

You can mark this as ready then.

Doh, forgot that it was in draft status. 🤦

@wraithgar wraithgar merged commit 7303db1 into npm:main Nov 30, 2023
26 checks passed
@github-actions github-actions bot mentioned this pull request Dec 4, 2023
wraithgar pushed a commit that referenced this pull request Feb 5, 2024
🤖 I have created a release *beep* *boop*
---


## [7.6.0](v7.5.4...v7.6.0)
(2024-01-31)

### Features

*
[`a7ab13a`](a7ab13a)
[#671](#671) preserve pre-release
and build parts of a version on coerce (#671) (@madtisa, madtisa,
@wraithgar)

### Chores

*
[`816c7b2`](816c7b2)
[#667](#667) postinstall for
dependabot template-oss PR (@lukekarrys)
*
[`0bd24d9`](0bd24d9)
[#667](#667) bump
@npmcli/template-oss from 4.21.1 to 4.21.3 (@dependabot[bot])
*
[`e521932`](e521932)
[#652](#652) postinstall for
dependabot template-oss PR (@lukekarrys)
*
[`8873991`](8873991)
[#652](#652) chore: chore:
postinstall for dependabot template-oss PR (@lukekarrys)
*
[`f317dc8`](f317dc8)
[#652](#652) bump
@npmcli/template-oss from 4.19.0 to 4.21.0 (@dependabot[bot])
*
[`7303db1`](7303db1)
[#658](#658) add clean() test for
build metadata (#658) (@jethrodaniel)
*
[`6240d75`](6240d75)
[#656](#656) add missing quotes
in README.md (#656) (@zyxkad)
*
[`14d263f`](14d263f)
[#625](#625) postinstall for
dependabot template-oss PR (@lukekarrys)
*
[`7c34e1a`](7c34e1a)
[#625](#625) bump
@npmcli/template-oss from 4.18.1 to 4.19.0 (@dependabot[bot])
*
[`123e0b0`](123e0b0)
[#622](#622) postinstall for
dependabot template-oss PR (@lukekarrys)
*
[`737d5e1`](737d5e1)
[#622](#622) bump
@npmcli/template-oss from 4.18.0 to 4.18.1 (@dependabot[bot])
*
[`cce6180`](cce6180)
[#598](#598) postinstall for
dependabot template-oss PR (@lukekarrys)
*
[`b914a3d`](b914a3d)
[#598](#598) bump
@npmcli/template-oss from 4.17.0 to 4.18.0 (@dependabot[bot])

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants