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

Fix lint #363

Merged
merged 1 commit into from
Jul 22, 2024
Merged

Fix lint #363

merged 1 commit into from
Jul 22, 2024

Conversation

etiennebacher
Copy link
Contributor

I was just testing flint on real packages and found this one, might as well fix it ;)

@CGMossa
Copy link
Member

CGMossa commented Jul 18, 2024

Flint is already providing results!

@CGMossa
Copy link
Member

CGMossa commented Jul 18, 2024

@Ilia-Kosenkov CI is failing here due to something about license note

══ Failed tests ════════════════════════════════════════════════════════════════
── Failure ('test-license_note.R:9:3'): LICENSE.note is generated properly ─────
Snapshot of code has changed:
old[15:21] vs new[15:21]
    
    Name:        extendr-macros
    Repository:  https://github.com/extendr/extendr
-   Authors:     andy-thomason, Thomas Down, Mossa Merhi Reimert, Claus O. Wilke, Hiroaki Yutani, Ilia A. Kosenkov
+   Authors:     andy-thomason, Thomas Down, Mossa Merhi Reimert, Claus O. Wilke, Hiroaki Yutani, Ilia A. Kosenkov, Michael Milton
    License:     MIT
    
    -------------------------------------------------------------

* Run `testthat::snapshot_accept('license_note')` to accept the change.
Error: Error: R CMD check found ERRORs
* Run `testthat::snapshot_review('license_note')` to interactively review the change.

@Ilia-Kosenkov Ilia-Kosenkov mentioned this pull request Jul 21, 2024
@CGMossa
Copy link
Member

CGMossa commented Jul 21, 2024

Rebased. Should work now as the error in CI was fixed in #365. Approving when test is passed, because we have auto-merge on.

@Ilia-Kosenkov Ilia-Kosenkov enabled auto-merge (squash) July 22, 2024 06:18
@Ilia-Kosenkov Ilia-Kosenkov merged commit c012ad8 into extendr:main Jul 22, 2024
17 checks passed
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