question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Separate rules into individual files

See original GitHub issue

I’ve hit an issue with my contribution on https://github.com/sqlfluff/sqlfluff/pull/577 (L035) where git was mixing up my issue with a newer issue (L036) when rebasing my branch.

Another issue that came up is that another rule https://github.com/sqlfluff/sqlfluff/pull/678 was started with the identifier L035 which is technically free since my PR was not merged yet. One of those contributions will have to be changed to L037 and then the actual code needs to be ordered in the file properly so L035 comes before L036 and then L037

I think we should just break-up the std.py file into one file per rule just like we did for tests to avoid these issues.

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Comments:11 (11 by maintainers)

github_iconTop GitHub Comments

1reaction
barrywhartcommented, Jan 13, 2021

I agree – std.py has become way too large: about 3,500 lines and counting. I’m happy to rename “my” L035 if needed.

0reactions
barrywhartcommented, Jan 16, 2021

That approach should work fine. The important thing about auto discovery is that it probably sounds more complex than it is.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Single make rule for multiple output files in different directories
Pattern rules with multiple targets define a recipe that creates multiple output files when run one time. For this I'd just write the...
Read more >
Import or export a set of rules - Microsoft Support
Click Import Rules. · Select the rules file that you want to import. · Click Open.
Read more >
How to split a PDF file - Adobe Support
Click the Split button, then click OK again​​ Click the Split button, and then click OK when Acrobat provides confirmation that the PDF...
Read more >
Publication 504 (2021), Divorced or Separated Individuals - IRS
This publication explains tax rules that apply if you are divorced or separated from your spouse. It covers general filing information and can...
Read more >
Snakefiles and Rules — Snakemake 7.19.1 documentation
Snakefiles and Rules¶. A Snakemake workflow defines a data analysis in terms of rules that are specified in the Snakefile. Most commonly, rules...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found