Skip to content

Switching to dibber#62

Merged
lietu merged 6 commits into
masterfrom
feature/dibber
Sep 12, 2025
Merged

Switching to dibber#62
lietu merged 6 commits into
masterfrom
feature/dibber

Conversation

@lietu
Copy link
Copy Markdown

@lietu lietu commented Sep 11, 2025

Switched from locally hosted code for the builder to using https://pypi.org/project/dibber/

  • Updated pre-commit config
  • Updated README
  • Updated LICENSE
  • Updated configs for bandit, prettier, etc.
  • Updated actions

Tested locally with dibber build and it completed successfully.

Comment thread .prettierrc.toml Outdated
Comment thread .github/workflows/build-and-upload.yaml Outdated
Comment thread .github/workflows/build-and-upload.yaml Outdated
Comment thread .github/workflows/build-and-upload.yaml Outdated
Comment thread .github/workflows/keepalive.yaml Outdated
Comment thread .github/workflows/scan-images.yaml Outdated
Comment thread .github/workflows/scan-images.yaml Outdated
Comment thread .github/workflows/pre-commit.yaml Outdated
Comment thread .github/workflows/pre-commit.yaml Outdated
Comment thread bandit.yaml Outdated
Copy link
Copy Markdown

@joakimnordling joakimnordling left a comment

Choose a reason for hiding this comment

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

In general looks very good! I added some small nitpicking details; most of them can likely just be applied from the comments.

lietu and others added 3 commits September 11, 2025 20:56
Co-authored-by: Joakim Nordling <joakim.nordling@ioxio.com>
Co-authored-by: Joakim Nordling <joakim.nordling@ioxio.com>
Copy link
Copy Markdown

@joakimnordling joakimnordling left a comment

Choose a reason for hiding this comment

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

Thanks for the quick changes, looks awesome!

@lietu lietu merged commit 32e0e6e into master Sep 12, 2025
1 of 2 checks passed
@lietu lietu deleted the feature/dibber branch September 12, 2025 07:18
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.

2 participants