Skip to content

Maintenance: tracer fix linting #289

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.

Sign up for GitHub

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

Jump to bottom
Closed
dreamorosi opened this issue Dec 10, 2021 · 2 comments
Closed

Maintenance: tracer fix linting #289

dreamorosi opened this issue Dec 10, 2021 · 2 comments
Assignees
@dreamorosi
Labels
completed This item is complete and has been merged/shipped good-first-issue Something that is suitable for those who want to start contributing internal PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.) tracer This item relates to the Tracer Utility
Milestone
production-ready-...

Comments

@dreamorosi
Copy link
Contributor

dreamorosi commented Dec 10, 2021 •
edited
Loading

Description

Implement linting for tracer module following what done for logger in #247 ; should be implemented after #107 has been merged.

Expected Behavior

New configuration should be present & npm run lint should be successful.

Current Behavior

Linting fails which causes automated checks to fail as well.
@dreamorosi dreamorosi added enhancement good-first-issue Something that is suitable for those who want to start contributing tracer This item relates to the Tracer Utility labels Dec 10, 2021
@dreamorosi dreamorosi added this to the production-ready-release milestone Dec 10, 2021
@dreamorosi dreamorosi self-assigned this Dec 15, 2021
@dreamorosi
Copy link
Contributor Author

Linting is now consistent with rest of the packages.

@github-actions
Copy link
Contributor

⚠️ COMMENT VISIBILITY WARNING ⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

@dreamorosi dreamorosi changed the title (tracer): fix linting Maintenance: tracer fix linting Nov 14, 2022
@dreamorosi dreamorosi added internal PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.) completed This item is complete and has been merged/shipped labels Nov 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dreamorosi dreamorosi

Labels
completed This item is complete and has been merged/shipped good-first-issue Something that is suitable for those who want to start contributing internal PRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.) tracer This item relates to the Tracer Utility
Projects
None yet
Milestone
production-ready-release
Development

No branches or pull requests
1 participant
@dreamorosi