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.

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

Update the set of linters #200

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Update the set of linters #200

wants to merge 5 commits into from

Conversation

RasmusSkytte
Copy link
Contributor

@RasmusSkytte RasmusSkytte commented Jan 10, 2025 •
edited
Loading

Intent

Sync the linters and reformat the code accordingly

Approach

N/A

Known issues

It seems we need SCDB v0.5 before this will work fully
(This changes SCDB::nrow() to always return integers)

Checklist

  • The PR passes all local unit tests
  • I have documented any new features introduced
  • If the PR adds a new feature, please add an entry in NEWS.md
  • A reviewer is assigned to this PR

@RasmusSkytte RasmusSkytte changed the base branch from repo-sync/AEF-DDF/default to main January 10, 2025 13:38
@RasmusSkytte RasmusSkytte self-assigned this Jan 10, 2025
@RasmusSkytte RasmusSkytte marked this pull request as ready for review January 10, 2025 14:14
@RasmusSkytte RasmusSkytte marked this pull request as draft January 10, 2025 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@RasmusSkytte RasmusSkytte

Labels
sync
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@RasmusSkytte