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: update FUSION_LOG_LEVEL configuration in vite config generation #2107

Merged
merged 1 commit into from
Apr 29, 2024

Conversation

odinr
Copy link
Collaborator

@odinr odinr commented Apr 29, 2024

Why

missing process env in production building of applications

closes: https://github.com/equinor/fusion/issues/343

Check off the following:

  • Confirm that I checked changes to branch which I am merging into.

    • I have validated included files
    • My code does not generate new linting warnings
    • My PR is not a duplicate, check existing pr`s
  • Confirm that the I have completed the self-review checklist.

  • Confirm that my changes meet our code of conduct.

Copy link

changeset-bot bot commented Apr 29, 2024

🦋 Changeset detected

Latest commit: 3e2b43a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 23 packages
Name Type
@equinor/fusion-query Patch
@equinor/fusion-framework-cli Patch
@equinor/fusion-framework-module-app Patch
@equinor/fusion-framework-module-bookmark Patch
@equinor/fusion-framework-module-context Patch
@equinor/fusion-framework-module-service-discovery Patch
@equinor/fusion-framework-module-widget Patch
@equinor/fusion-framework-react-components-people-provider Patch
@equinor/fusion-framework-react-module-bookmark Patch
@equinor/fusion-framework-react-module-context Patch
@equinor/fusion-framework-cookbook-app-react-context-custom-error Patch
@equinor/fusion-framework-cookbook-app-react-context Patch
@equinor/fusion-framework-cookbook-app-react-feature-flag Patch
@equinor/fusion-framework-app Patch
@equinor/fusion-framework-react-app Patch
@equinor/fusion-framework-react-components-bookmark Patch
@equinor/fusion-framework-react Patch
@equinor/fusion-framework-legacy-interopt Patch
@equinor/fusion-framework Patch
@equinor/fusion-framework-module-signalr Patch
@equinor/fusion-framework-react-widget Patch
@equinor/fusion-framework-widget Patch
@equinor/fusion-framework-react-module-signalr Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@odinr odinr self-assigned this Apr 29, 2024
@github-actions github-actions bot added 🐞 bug Something isn't working 💾 CLI fusion framework CLI 📚 documentation Improvements or additions to documentation 🛠️ utils packages related to utils labels Apr 29, 2024
@odinr
Copy link
Collaborator Author

odinr commented Apr 29, 2024

@eikeland can you checkout branch, build the context cookbook (fusion-framework-cli app pack) and test it in a dev-portal.
Also check with fusion-framework-cli app dev and verify that local log is set to '3'

@odinr odinr marked this pull request as ready for review April 29, 2024 08:54
@odinr odinr requested a review from a team as a code owner April 29, 2024 08:54
Copy link
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 11.25% 3459 / 30735
🔵 Statements 11.25% 3459 / 30735
🔵 Functions 11.02% 85 / 771
🔵 Branches 24.06% 207 / 860
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
packages/cli/src/lib/vite-config.ts 0% 0% 0% 0% 1-114
packages/utils/query/src/logger.ts 96.8% 58.33% 85.71% 96.8% 47, 49, 115-116
Generated in workflow #5748

@odinr odinr merged commit 491c2e0 into main Apr 29, 2024
14 checks passed
@odinr odinr deleted the fix/set-global-process-envs branch April 29, 2024 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug Something isn't working 💾 CLI fusion framework CLI 📚 documentation Improvements or additions to documentation 🛠️ utils packages related to utils
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants