Skip to content

Appease lints given Jarl update - #247

Merged
matt-dray merged 3 commits into
mainfrom
246-lint
Aug 27, 2026
Merged

matt-dray merged 3 commits into
mainfrom
246-lint

Conversation

@matt-dray

@matt-dray matt-dray commented Aug 25, 2026 •

Copy link
Copy Markdown
Contributor

Close #246. One for you, @Lextuga007?

Note

Update to Jarl v0.6.0 if running locally.

  • Remove trailing commas.
  • Add jarl.toml config file to ignore empty, but required, R/_disable_autoload.R.
  • Bump to dev version 0.8.0.9000.

@matt-dray matt-dray self-assigned this Aug 25, 2026
@matt-dray matt-dray added this to the v0.8.1 milestone Aug 25, 2026
@matt-dray matt-dray added bug Something isn't working priority: should MoSCoW priority labels Aug 25, 2026
@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (aae9704) to head (27b4090).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #247   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           27        27           
  Lines         1294      1294           
=========================================
  Hits          1294      1294           

☔ View full report in Codecov by Harness.
📢 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.

@Lextuga007 Lextuga007 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Locally I added:

x = data.frame(a = 1, a = 1)

to /_disable_autoload.R and ran jarl check . from the command line - all checks passed! Then added the line to app_ui.R where the line was flagged as "Avoid duplicated arguments..." as expected.

@matt-dray
matt-dray merged commit 4a7808b into main Aug 27, 2026
6 checks passed
@matt-dray
matt-dray deleted the 246-lint branch August 27, 2026 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority: should MoSCoW priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Respond to new Jarl lint updates

2 participants