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

build: use workspaces #989

Merged

Conversation

blaggacao
Copy link
Contributor

@blaggacao blaggacao commented Aug 2, 2024

Like other Frappe apps.

This, at the very least, avoids build issues (to be documented below):

This blocks: blaggacao/frappix#10

nikkothari22 and others added 3 commits July 26, 2024 20:50
@blaggacao blaggacao changed the base branch from develop to main August 2, 2024 08:53
@blaggacao blaggacao marked this pull request as draft August 2, 2024 08:54
@blaggacao blaggacao marked this pull request as ready for review August 2, 2024 09:17
@nikkothari22 nikkothari22 changed the base branch from main to develop August 2, 2024 09:20
@blaggacao blaggacao changed the base branch from develop to main August 2, 2024 09:23
@nikkothari22
Copy link
Member

@blaggacao The base branch is "develop". Not main. We'll merge develop into main when we make a new release (which should be today)

@nikkothari22 nikkothari22 changed the base branch from main to develop August 2, 2024 09:47
@nikkothari22
Copy link
Member

Ah it looks like you created the branch from main.

@nikkothari22 nikkothari22 changed the base branch from develop to main August 2, 2024 09:51
@nikkothari22 nikkothari22 merged commit bba550e into The-Commit-Company:main Aug 2, 2024
3 checks passed
@blaggacao
Copy link
Contributor Author

Nice! Thanks a lot, your reaction time is refreshing and much appreciated. Thank you! 🤝

@blaggacao blaggacao deleted the build/use-workspaces branch August 2, 2024 09:56
@nikkothari22
Copy link
Member

Looks like builds are failing on Frappe Cloud :(

image

Might be related to this?

@nikkothari22
Copy link
Member

Fixed the build issue via: 25a1b8c . Missed the extra workspace here.

@blaggacao
Copy link
Contributor Author

Ah! I'm sorry for the oversight. It had a package.json, so I heuristically added it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants