Skip to content

Doc load logs#1711

Merged
aidanm3341 merged 4 commits intofinos:mainfrom
willosborne:doc-load-logs
Nov 13, 2025
Merged

Doc load logs#1711
aidanm3341 merged 4 commits intofinos:mainfrom
willosborne:doc-load-logs

Conversation

@willosborne
Copy link
Member

Description

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📚 Documentation update
  • 🎨 Code style/formatting changes
  • ♻️ Refactoring (no functional changes)
  • ⚡ Performance improvements
  • ✅ Test additions or updates
  • 🔧 Chore (maintenance, dependencies, CI, etc.)

Affected Components

  • CLI (cli/)
  • Shared (shared/)
  • CALM Widgets (calm-widgets/)
  • CALM Hub (calm-hub/)
  • CALM Hub UI (calm-hub-ui/)
  • Documentation (docs/)
  • VS Code Extension (calm-plugins/vscode/)
  • Dependencies
  • CI/CD

Commit Message Format ✅

Testing

  • I have tested my changes locally
  • I have added/updated unit tests
  • All existing tests pass

Checklist

  • My commits follow the conventional commit format
  • I have updated documentation if necessary
  • I have added tests for my changes (if applicable)
  • My changes follow the project's coding standards

aidanm3341
aidanm3341 previously approved these changes Oct 14, 2025
@rocketstack-matt
Copy link
Member

Assume this is related to #1681 ?

I see we're still having the CVE scans fail 😢 will have to try some more things to see if I can get it working now it requires authentication, in the meantime I see Aidan approved but we have a conflict in the package-lock.json. Are you able to resolve that and we will merge.

@markscott-ms
Copy link
Contributor

@willosborne I suspect your significant change to package-lock.json has broken the builds.

package-lock.json: 1,585 additions, 6,385 deletions

@willosborne
Copy link
Member Author

@markscott-ms all i did was a fresh npm install - i havent changed the dependencies at all! i don't understand why this doesn't work. and i have to do it, because otherwise there are conflicts

@willosborne
Copy link
Member Author

let me try removing node modules

@willosborne
Copy link
Member Author

aha! deleting node_modules before an npm install fixed it...

@aidanm3341 aidanm3341 merged commit c5cac37 into finos:main Nov 13, 2025
12 checks passed
@willosborne willosborne deleted the doc-load-logs branch November 13, 2025 15:33
pmerrison pushed a commit to pmerrison/architecture-as-code that referenced this pull request Nov 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants