Skip to content

fix npm audit warnings#2196

Closed
xtofs wants to merge 4 commits into
oasis-tcs:mainfrom
xtofs:main
Closed

fix npm audit warnings#2196
xtofs wants to merge 4 commits into
oasis-tcs:mainfrom
xtofs:main

Conversation

@xtofs
Copy link
Copy Markdown

@xtofs xtofs commented Apr 7, 2026

No description provided.

HeikoTheissen
HeikoTheissen previously approved these changes Apr 9, 2026
@HeikoTheissen
Copy link
Copy Markdown
Contributor

Merge this and close #2188

@ralfhandl
Copy link
Copy Markdown
Contributor

@xtofs Can you please check if actions are enabled in your repo fork?

@chrisspre
Copy link
Copy Markdown

@xtofs Can you please check if actions are enabled in your repo fork?

they are not (yet)

@ralfhandl
Copy link
Copy Markdown
Contributor

Could you please enable them? The build action needs to run successfully before this PR can be merged.

@xtofs
Copy link
Copy Markdown
Author

xtofs commented Apr 13, 2026

Could you please enable them? The build action needs to run successfully before this PR can be merged.

enabled and ran sucessfully https://github.com/xtofs/odata-abnf/actions/runs/24344651239

I only see one workflow. So I am not sure if that is complete.

@HeikoTheissen
Copy link
Copy Markdown
Contributor

enabled and ran sucessfully https://github.com/xtofs/odata-abnf/actions/runs/24344651239

Wrong repo. This is about https://github.com/xtofs/odata-specs.

Comment thread .github/workflows/nodejs.yml Outdated
on: push
on:
push:
workflow_dispatch:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why is this necessary?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

not strictly necessary. I added this so i can run this workflow manually in my fork.
But I am removing it now since it did it's purpose

Removed 'workflow_dispatch' trigger from Node CI workflow.
@xtofs xtofs closed this by deleting the head repository Apr 25, 2026
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