Skip to content

Fix: Errors as per linter logs in PR#89#104

Merged
dlebauer merged 1 commit into
PecanProject:masterfrom
Mansi-dass:mansi-pylint
Dec 1, 2025
Merged

Fix: Errors as per linter logs in PR#89#104
dlebauer merged 1 commit into
PecanProject:masterfrom
Mansi-dass:mansi-pylint

Conversation

@Mansi-dass
Copy link
Copy Markdown
Contributor

@Mansi-dass Mansi-dass commented May 11, 2023

The log errors are resolved by updating the template provided by super-linter ,

As the template file has gone through some latest changes -

  • A lot of identifiers are removed from disable category (because of which the error in logs)
  • code blocks are rearranged, so that's why so many diffs
  • compared the previous template and the new template, for any customization in the previous one and also added that to the new template.

Resolves issue: #70
Fix errors based on PR: #89

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 11, 2023

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 6.53%. Comparing base (b31e02d) to head (6d6f6a7).
⚠️ Report is 10 commits behind head on master.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@          Coverage Diff           @@
##           master    #104   +/-   ##
======================================
  Coverage    6.53%   6.53%           
======================================
  Files          20      20           
  Lines         597     597           
  Branches       68      68           
======================================
  Hits           39      39           
  Misses        506     506           
  Partials       52      52           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Mansi-dass Mansi-dass changed the title Resolved errors as per linter logs in PR#89 Fix: Errors as per linter logs in PR#89 Jul 20, 2023
@dlebauer dlebauer merged commit 6d91142 into PecanProject:master Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants