Skip to content

Conversation

@azimgd
Copy link
Contributor

@azimgd azimgd commented Feb 6, 2023

(electron) webpack build includes const file during compilation, hence build fails when document is used outside of main process.

@azimgd
Copy link
Contributor Author

azimgd commented Feb 6, 2023

luacmartins
luacmartins previously approved these changes Feb 6, 2023
Copy link
Contributor

@luacmartins luacmartins left a comment

Choose a reason for hiding this comment

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

@azimgd can you actually bump the package version as well?

@luacmartins luacmartins self-requested a review February 6, 2023 19:54
@azimgd
Copy link
Contributor Author

azimgd commented Feb 6, 2023

Sure. Btw, are you using: npm run release for deployments, it should automate everything for you ?

@azimgd
Copy link
Contributor Author

azimgd commented Feb 6, 2023

@luacmartins done.

@luacmartins
Copy link
Contributor

luacmartins commented Feb 6, 2023

@azimgd I'm not. We usually follow this process to release our packages, or we built a GH action like this one to automate the publishing process.

@luacmartins luacmartins merged commit 879d100 into Expensify:main Feb 6, 2023
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