Skip to content

feat: barebones builds #10

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jan 8, 2025
Merged

feat: barebones builds #10

merged 4 commits into from
Jan 8, 2025

Conversation

bpevs
Copy link
Owner

@bpevs bpevs commented Jan 7, 2025

Basically, until now Bext technically worked for minimal web extensions, but created a lot of excess build files and had some opinions about structure. So this PR (and also #9) address issues such as #8, to allow for more flexibility in application structure.

The changes introduces in this PR:

  • main.ts
    • Only copy the static dir if existing in source
    • Allow for using any combination of js/ts/jsx/tsx file endings, as well as allowing for camel-case contentScript
    • Only copy existing js/ts/jsx/tsx files if existing in source
  • examples
    • Add examples/barebones example to remove everything but a basic content_script
    • Adjust examples/preact manifest/code/icons to better distinguish from barebones
    • fixes an issue with the examples/preact css, because wtf is that?

bpevs added 4 commits January 7, 2025 20:41
- only copy static dir if existing
- only copy existing source files
- add barebones example
- adjust preact meta to distinguish from barebones
Copy link
Contributor

@moritztim moritztim left a comment

Choose a reason for hiding this comment

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

Looks good to me, aside from that one thing.

@bpevs bpevs merged commit 53fe34a into main Jan 8, 2025
2 checks passed
@bpevs bpevs deleted the barebones branch January 8, 2025 13:37
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