Skip to content
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

feat: add build process via rollup #21

Merged
merged 7 commits into from
Nov 30, 2024
Merged

feat: add build process via rollup #21

merged 7 commits into from
Nov 30, 2024

Conversation

bdbch
Copy link
Member

@bdbch bdbch commented Nov 30, 2024

Added a build process via Rollup, tagged the core version to alpha, moved the @atproto package to a dependency and tested the build process.

Copy link

netlify bot commented Nov 30, 2024

Deploy Preview for tsky canceled.

Name Link
🔨 Latest commit 55d1974
🔍 Latest deploy log https://app.netlify.com/sites/tsky/deploys/674af54df25cbc000850dea1

@anbraten
Copy link
Contributor

Using https://github.com/privatenumber/pkgroll might help with the rollup config

@bdbch
Copy link
Member Author

bdbch commented Nov 30, 2024

Using https://github.com/privatenumber/pkgroll might help with the rollup config

Cool stuff. Will give it a shot.

@bdbch
Copy link
Member Author

bdbch commented Nov 30, 2024

rollpkg is awesome - used it! :)

@bdbch bdbch marked this pull request as ready for review November 30, 2024 11:22
- name: 📦 Build
run: pnpm -r build

# - name: 🚢 Continuous Release
Copy link
Contributor

Choose a reason for hiding this comment

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

An admin would need to install the app right?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep - already requested 😂

We can comment that out as soon as it's enable for this repo.

@bdbch bdbch merged commit 62984bb into main Nov 30, 2024
5 of 6 checks passed
@bdbch bdbch deleted the build branch November 30, 2024 11:28
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