Skip to content

Bugfix/spline 1124 foxx deploy string replace#1125

Merged
wajda merged 3 commits into
developfrom
bugfix/spline-1124-foxx-deploy-string-replace
Nov 7, 2022
Merged

Bugfix/spline 1124 foxx deploy string replace#1125
wajda merged 3 commits into
developfrom
bugfix/spline-1124-foxx-deploy-string-replace

Conversation

@wajda

@wajda wajda commented Nov 4, 2022

Copy link
Copy Markdown
Contributor

fixes #1124

  1. Got rid of custom file copying and replacing process. Re-implemented entire Foxx bundle build process using Webpack
  2. Suppressed unnecessary console log in the common.ts

@wajda wajda marked this pull request as ready for review November 4, 2022 16:40
@wajda wajda requested a review from cerveada as a code owner November 4, 2022 16:40
@wajda wajda requested review from dk1844 and korel-san November 4, 2022 16:40
@wajda wajda marked this pull request as draft November 4, 2022 16:57
@wajda wajda force-pushed the bugfix/spline-1124-foxx-deploy-string-replace branch from 2d14757 to f4e1b7f Compare November 4, 2022 17:26
@wajda wajda marked this pull request as ready for review November 4, 2022 17:27
@sonarqubecloud

sonarqubecloud Bot commented Nov 4, 2022

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@dk1844 dk1844 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested on Windows 10:
✔️ in spline\arangodb-foxx-services\target\dist, the version, revision and timestamp strings are correctly replaced in README.md and manifest.json
✔️ the above is then correctly reflected in the package spline\arangodb-foxx-services\target\arangodb-foxx-services-1.0.0-SNAPSHOT-dist.zip

@wajda wajda merged commit 8d69974 into develop Nov 7, 2022
@wajda wajda deleted the bugfix/spline-1124-foxx-deploy-string-replace branch November 7, 2022 11:46
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.

arangodb-foxx-services :: npm run foxx:deploy modifies source files

3 participants