Skip to content

Conversation

@char0n
Copy link
Contributor

@char0n char0n commented May 29, 2024

No description provided.

@char0n char0n self-assigned this May 29, 2024
@char0n char0n added enhancement New feature or request ApiDOM dependencies labels May 29, 2024
@char0n char0n merged commit f076207 into main May 29, 2024
@char0n char0n deleted the char0n/reference-error-as-dep branch May 29, 2024 20:00
"dependencies": {
"@babel/runtime-corejs3": "^7.20.7",
"@swagger-api/apidom-core": "^1.0.0-alpha.3",
"@swagger-api/apidom-error": "^1.0.0-alpha.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

@char0n Should @swagger-api/apidom-error have be be removed from the optionalDependencies below with this change?

Copy link
Contributor

Choose a reason for hiding this comment

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

Raised: #4864

leipert added a commit to leipert/apidom that referenced this pull request Apr 28, 2025
In swagger-api#4154 `apidom-error` was
added as a direct dependency, but wasn't removed as an optional
dependency. Given it is now required directly, let's just drop it from
the optional ones.
leipert added a commit to leipert/apidom that referenced this pull request Apr 28, 2025
In swagger-api#4154 `apidom-error` was
added as a direct dependency, but wasn't removed as an optional or dev
dependency. Given it is now required directly, let's just drop it from
the optional and the dev lists.
char0n pushed a commit that referenced this pull request Apr 28, 2025
Some time back `apidom-error` was added as a direct dependency, 
but wasn't removed as an optional or dev dependency. Given it is now
required directly, let's just drop it from the optional and the dev lists.

Refs #4154
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants