Skip to content

fix: 'axios' module not found locally#7638

Merged
sid-bruno merged 4 commits intousebruno:mainfrom
sid-bruno:fix/ssl-tests
Mar 31, 2026
Merged

fix: 'axios' module not found locally#7638
sid-bruno merged 4 commits intousebruno:mainfrom
sid-bruno:fix/ssl-tests

Conversation

@sid-bruno
Copy link
Copy Markdown
Collaborator

@sid-bruno sid-bruno commented Mar 31, 2026

Description

Contribution Checklist:

  • I've used AI significantly to create this pull request
  • The pull request only addresses one issue or adds one feature.
  • The pull request does not introduce any breaking changes
  • I have added screenshots or gifs to help explain the change if applicable.
  • I have read the contribution guidelines.
  • Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.

Publishing to New Package Managers

Please see here for more information.

Summary by CodeRabbit

  • Chores
    • Reduced the minimum release age configuration from 30 to 10.
    • Updated the Rollup override to a newer patch version to align build tooling.

@sid-bruno sid-bruno requested a review from helloanoop as a code owner March 31, 2026 13:50
Copilot AI review requested due to automatic review settings March 31, 2026 13:50
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 31, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 690b6811-d5ab-46b8-b704-17577c878e99

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

Updated two configuration/manifest entries: .npmrc changed min-release-age from 30 to 10, and packages/bruno-graphql-docs/package.json bumped the Rollup override from 3.29.5 to 3.30.0.

Changes

Cohort / File(s) Summary
NPM Configuration
/.npmrc
Single-line change: min-release-age updated from 3010.
Package manifest (Rollup override)
packages/bruno-graphql-docs/package.json
Dependency override: rollup version changed from 3.29.53.30.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

size/S

Suggested reviewers

  • helloanoop
  • lohit-bruno
  • naman-bruno
  • bijin-bruno

Poem

⚡ Down from thirty to ten we go,
📦 A tiny tweak to speed the flow,
🔧 One manifest saw Rollup rise,
🚀 Small changes, quiet skies — repose.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title claims to fix 'axios module not found locally', but the changeset only updates .npmrc min-release-age and rollup version in package.json—neither change addresses axios or module resolution for axios. Update the title to accurately reflect the actual changes: something like 'chore: update min-release-age and rollup dependency version' or clarify what axios issue these changes actually resolve.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sid-bruno sid-bruno changed the title fix fix: 'axios' module not found locally Mar 31, 2026
@sid-bruno sid-bruno merged commit 4a78f63 into usebruno:main Mar 31, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants