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

fix: export CFM version number rather than __PACKAGE_VERSION__ #357

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

kswenson
Copy link
Member

@kswenson kswenson commented Feb 21, 2024

The CFM code contains the string __PACKAGE_VERSION__ in several places. The expectation is that this string will be replaced by the actual CFM version number from package.json at build time. For the webpack build, this is handled by the ReplaceInFileWebpackPlugin. For the library export we don't have a bundler that can handle the string replacement. Therefore, this PR introduces the build-library.sh bash script which builds the exported library folders with calls to tsc, as before, then also uses grep and sed to perform the string replacement of the version number. The script was developed with the help of ChatGPT.

@kswenson kswenson force-pushed the export-lib-version branch 2 times, most recently from 9285471 to a577ac6 Compare February 21, 2024 00:57
@kswenson kswenson requested a review from dougmartin February 21, 2024 01:03
@kswenson kswenson changed the title fix: create a copy for CODAP v3 fix: export CFM version number rather than __PACKAGE_VERSION__ Feb 21, 2024
@kswenson kswenson requested review from eireland and removed request for dougmartin February 21, 2024 15:59
Copy link
Contributor

@eireland eireland left a comment

Choose a reason for hiding this comment

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

👍 LGTM

Base automatically changed from support-codap-v3-copy to master February 21, 2024 18:13
@kswenson kswenson merged commit 73b0cbb into master Feb 21, 2024
2 checks passed
@kswenson kswenson deleted the export-lib-version branch February 21, 2024 18:21
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