Skip to content

chore: publishing and modernization - #8

Merged
vknabel merged 5 commits into
mainfrom
update-js-and-templates
Sep 7, 2026
Merged

chore: publishing and modernization#8
vknabel merged 5 commits into
mainfrom
update-js-and-templates

Conversation

@vknabel

@vknabel vknabel commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

@vknabel
vknabel force-pushed the update-js-and-templates branch from aa610ad to fc25ca6 Compare September 3, 2026 12:23
@majst01

majst01 commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

test- and log-interceptor is missing

@vknabel
vknabel force-pushed the update-js-and-templates branch 2 times, most recently from b7cb259 to a1436d1 Compare September 3, 2026 12:28
@vknabel vknabel self-assigned this Sep 4, 2026

@simcod simcod left a comment

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.

We could also use the shorter configuration for the tsconfig.json as in the metal-stack/api including bun types.

Also add a build step for js, for reference see: https://github.com/metal-stack/api/blob/main/.github/workflows/main.yml#L37

Comment thread .github/workflows/npm-publish.yaml Outdated
Comment thread .github/workflows/npm-publish.yaml
Comment thread .github/workflows/tests.yaml Outdated
Comment thread .github/workflows/tests.yaml Outdated
Comment thread .github/workflows/tests.yaml Outdated
Comment thread js/package.json Outdated
@vknabel
vknabel force-pushed the update-js-and-templates branch from a1436d1 to 6f0d391 Compare September 7, 2026 07:48
@vknabel
vknabel force-pushed the update-js-and-templates branch from 6f0d391 to 4aa9573 Compare September 7, 2026 08:00
Co-authored-by: Simon Mayer <49491825+simcod@users.noreply.github.com>
Comment thread .github/workflows/main.yml Outdated
@majst01

majst01 commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

I think that we can skip the push to pypi.org for now because we do not have any ansible integration requirements atm

@vknabel vknabel changed the title chore: update chore: publishing and modernization Sep 7, 2026
@vknabel
vknabel marked this pull request as ready for review September 7, 2026 09:00
@vknabel
vknabel requested a review from majst01 September 7, 2026 09:00
@vknabel
vknabel merged commit fded87c into main Sep 7, 2026
8 checks passed
@vknabel
vknabel deleted the update-js-and-templates branch September 7, 2026 09:07
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.

Publish NPM Package

3 participants