Skip to content

Conversation

@savetheclocktower
Copy link
Collaborator

I started this PR because I noticed a broken link on the Basics page; then I decided to rewrite the web.pulsar-edit.dev URLs while I was here. Then I noticed something else:

We use registry and repository interchangeably, but they're different. What we're hosting is a registry — works almost exactly like NPM's package registry, but has Pulsar-specific packages.

It could be argued that it's also a “repository” of packages — not in the git sense, but just in the sense of the common noun. But “registry” functions equally well whether or not you know it's an NPM term of art, and doesn't run the risk of getting confused with any git terms of art.

So this PR also changes all usages of “package repository“ to be “package registry.”

…so that all the links to the package registry use the new URLs, and all references to the registry call it the same thing.

We use `registry` and `repository` interchangeably, but they're different. What we're hosting is a registry — works almost exactly like NPM's package registry, but has Pulsar-specific packages.

It could be argued that it's also a “repository” of packages — not in the `git` sense, but just in the sense of the common noun. But “registry” functions equally well whether or not you know it's an NPM term of art, and doesn't run the risk of getting confused with any `git` terms of art.
Copy link
Member

@confused-Techie confused-Techie left a comment

Choose a reason for hiding this comment

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

I fully approve the content of this PR, but before merged it seems you've accidentally left a digested.json in the diff from sometime when running the API creation script.

Would you mind deleting that file from this branch before merging this one?

Otherwise the changes look good

@savetheclocktower
Copy link
Collaborator Author

Would you mind deleting that file from this branch before merging this one?

Good catch! PR updated.

@savetheclocktower savetheclocktower merged commit 67e10b7 into main Sep 7, 2025
@savetheclocktower savetheclocktower deleted the fix-broken-links branch September 7, 2025 21:47
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.

3 participants