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

Reduce default log output #125

Merged
merged 2 commits into from
Jan 31, 2023
Merged

Reduce default log output #125

merged 2 commits into from
Jan 31, 2023

Conversation

domjtalbot
Copy link
Owner

The initial log output is showing debug level messages.

What's Changed?

  1. The default output log only shows necessary information.
  2. Errors are shown in full.
  3. Debug mode enables all logs

@changeset-bot
Copy link

changeset-bot bot commented Jan 31, 2023

🦋 Changeset detected

Latest commit: a0b027d

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

This PR includes changesets to release 1 package
Name Type
nx-mesh 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

}
});

childProcess?.stderr?.on('data', (chunk) => {
Copy link

Choose a reason for hiding this comment

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

Identical blocks of code found in 2 locations. Consider refactoring.

}
});

childProcess?.stderr?.on('data', (chunk) => {
Copy link

Choose a reason for hiding this comment

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

Identical blocks of code found in 2 locations. Consider refactoring.

);

childProcess?.stdout?.on('data', (chunk) => {
Copy link

Choose a reason for hiding this comment

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

Identical blocks of code found in 2 locations. Consider refactoring.

);

childProcess?.stdout?.on('data', (chunk) => {
Copy link

Choose a reason for hiding this comment

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

Identical blocks of code found in 2 locations. Consider refactoring.

await watcher(
async () => {
await runMeshCli(
runMeshCli(
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 2 locations. Consider refactoring.

@codeclimate
Copy link

codeclimate bot commented Jan 31, 2023

Code Climate has analyzed commit a0b027d and detected 8 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 8

View more on Code Climate.

@domjtalbot domjtalbot self-assigned this Jan 31, 2023
@nx-cloud
Copy link

nx-cloud bot commented Jan 31, 2023

☁️ Nx Cloud Report

CI is running/has finished running commands for commit a0b027d. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch


✅ Successfully ran 23 targets

Sent with 💌 from NxCloud.

@domjtalbot domjtalbot temporarily deployed to example-sdk-nextjs-preview January 31, 2023 23:04 — with GitHub Actions Inactive
@github-actions github-actions bot temporarily deployed to example-sdk-nextjs-preview January 31, 2023 23:04 Inactive
@github-actions
Copy link
Contributor

🚀 Snapshot Release (alpha)

The latest changes of this PR are available as alpha on npm (based on the declared changesets):

Package Version Info
nx-mesh 3.1.0-alpha-20230131230422 npm ↗︎ unpkg ↗︎

@sonarcloud
Copy link

sonarcloud bot commented Jan 31, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

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

0.0% 0.0% Coverage
67.3% 67.3% Duplication

@github-actions
Copy link
Contributor

This pull request has been deployed to Vercel.

Latest commit: a0b027d
✅ Preview: https://nx-mesh-aqjptw2o1-domjtalbot.vercel.app
🔍 Inspect: https://vercel.com/domjtalbot/nx-mesh/4aYCJ9LZP3ZVFK4RHJ2tZbd9MbaP

View Workflow Logs

@github-actions github-actions bot added the vercel An issue or pull request related to Vercel label Jan 31, 2023
@domjtalbot domjtalbot merged commit 7e95367 into main Jan 31, 2023
@domjtalbot domjtalbot deleted the fix/reduce-default-log branch January 31, 2023 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vercel An issue or pull request related to Vercel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant