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

[ci/cd] Make the next branch easy to deploy #241

Merged
merged 31 commits into from
Aug 21, 2023
Merged

[ci/cd] Make the next branch easy to deploy #241

merged 31 commits into from
Aug 21, 2023

Conversation

mdartic
Copy link
Member

@mdartic mdartic commented Jul 19, 2023

No description provided.

@mdartic mdartic self-assigned this Jul 19, 2023
@mdartic mdartic force-pushed the 239-build branch 2 times, most recently from d0eadde to 9fe8a47 Compare August 2, 2023 15:48
@mdartic mdartic changed the title Make the next branch easy to deploy [ci/cd] Make the next branch easy to deploy Aug 2, 2023
@mdartic mdartic force-pushed the 239-build branch 7 times, most recently from 0a01e7d to 0053d31 Compare August 3, 2023 08:43
@codecov
Copy link

codecov bot commented Aug 3, 2023

Codecov Report

❗ No coverage uploaded for pull request base (next@8712f5d). Click here to learn what that means.
Patch has no changes to coverable lines.

❗ Current head ded8001 differs from pull request most recent head 48919b2. Consider uploading reports for the commit 48919b2 to get more accurate results

Additional details and impacted files
@@          Coverage Diff          @@
##             next   #241   +/-   ##
=====================================
  Coverage        ?      0           
=====================================
  Files           ?      0           
  Lines           ?      0           
  Branches        ?      0           
=====================================
  Hits            ?      0           
  Misses          ?      0           
  Partials        ?      0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mdartic mdartic force-pushed the 239-build branch 17 times, most recently from e91d1a2 to 8eb8485 Compare August 4, 2023 12:57
docker-compose.prod.yml Fixed Show fixed Hide fixed
@mdartic mdartic force-pushed the 239-build branch 5 times, most recently from 20a4d7e to 7dc489d Compare August 8, 2023 21:25
lck-api.env Outdated Show resolved Hide resolved
lck-api.env Outdated Show resolved Hide resolved
packages/designsystem/package.json Show resolved Hide resolved
packages/designsystem/package.json Show resolved Hide resolved
packages/designsystem/plugins/vee-validate.ts Show resolved Hide resolved
packages/nuxt-locokit/src/module.ts Outdated Show resolved Hide resolved
packages/nuxt-locokit/src/module.ts Show resolved Hide resolved
packages/nuxt-locokit/src/runtime/stores/auth.ts Outdated Show resolved Hide resolved
packages/nuxt-locokit/src/runtime/pages/auth/login.vue Outdated Show resolved Hide resolved
*/
addLayout(
{
src: resolve(layoutsDir, './WithHeader.vue'),
Copy link
Contributor

Choose a reason for hiding this comment

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

I didn't see change in file using layout. Is this normal?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes.
Adding layout was clearly breaking the build and usage of the nuxt module, as it was referencing files in the module.
This seems not OK, but I didn't find any documentation on this.

Pages using layouts inside the module are OK.
Pages in the app using our nuxt-locokit module won't have access to our layouts, until our layouts need other files of the module (like paths).

So, for now, I don't have other work around for this. I simply remove the addLayout.
And this is fine as we were not using layouts in our nuxt playground / app.

* remove log in nuxt module
* remove routes file in nuxt module
* remove auth login page (legacy code)
Base automatically changed from 234-qlty-ds to next August 11, 2023 15:17
@sonarcloud
Copy link

sonarcloud bot commented Aug 11, 2023

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 11 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mdartic mdartic merged commit 448c13b into next Aug 21, 2023
7 checks passed
@mdartic mdartic deleted the 239-build branch August 21, 2023 07:49
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.

[ci/cd] Automate the build of api, app, nuxt-module, engine and definition packages
2 participants