Skip to content

Conversation

TimvdLippe
Copy link

Tags added to methods are now automatically appended to the existing list of global tags.

Fixes #65

@TimvdLippe TimvdLippe requested a review from wesleytodd as a code owner April 18, 2025 08:42
.expect(200, (err, res) => {
assert(!err, err)
assert.strictEqual(Object.keys((res.body.paths))[0], '/{id}/')
assert.strictEqual(Object.keys((res.body.paths))[0], '(?:/([^/]+?))/')
Copy link
Author

Choose a reason for hiding this comment

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

Not sure why this test was complaining for me, but when I run npm test on main, it was failing. By updating it to this regex, it was passing for me again.

Copy link
Collaborator

Choose a reason for hiding this comment

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

This is failing for me too. I just created a pull request to fix it #74

@TimvdLippe
Copy link
Author

Not sure about Node 16 failure. They are failing for the validation tests, not the ones I added. Also, on main there was also a CI failure, so I think it is unrelated to the changes I made?

Tags added to methods are now automatically appended
to the existing list of global tags.

Fixes wesleytodd#65
@TimvdLippe TimvdLippe force-pushed the pr-timvdlippe-add-support-for-tags branch from c2806a6 to dfc0ef4 Compare April 18, 2025 09:05
@TimvdLippe
Copy link
Author

@wesleytodd @Megapixel99 PTAL, we would like to use this in our own example

@Megapixel99 Megapixel99 added enhancement New feature or request good first issue Good for newcomers labels Jun 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for tags
3 participants