Comments (9)
@nelson6e65 My bad, I looked. I'm using esbenp.prettier-vscode
. I guess we should just pick one and make it known so that it is easier to make small changes in PRs.
@ondras what do you think?
from wwwsqldesigner.
Ok, when I used the word clean, what I meant was small
or much smaller
See here #254, this was indicated as too big, I had a quick look and also felt it was beyond what I can afford, largely due to it being hard to read.
Reading to understand by itself is work :)
And the latest commit was only removing 5 lines, but there was another extra 16 additions and deletions (probably due to some auto formating of tabs to spaces etc)
4c2b6f5
It's just a suggestion, but would require core collaborators to stick to using the auto formatter, otherwise it would not bring any benefits. Thank you.
from wwwsqldesigner.
No worries, thanks! Now I can based my other PRs off this :)
from wwwsqldesigner.
VSCode? Did you mean ESLint / Prettier / PHPStan / PHPCS?
from wwwsqldesigner.
I am not sure this is going to solve a problem I am having. To address your arguments:
- Easier for anyone to file clean PRs
In what sense? I do not recall dismissing a PR due to it being "unclean".
2. Higher readability, less work
There is no work being mandated with respect to formatting. Obviously becoming consistent with the "code style status quo" is useful.
I am okay with a PR that somewhat normalizes whitespace (as long as it does not introduce style changes incompatible with my own coding style). But I do not plan on applying that manually and/or automatically to my further code base adjustments.
from wwwsqldesigner.
See here #254, this was indicated as too big, I had a quick look and also felt it was beyond what I can afford, largely due to it being hard to read.
Right, but the related functionality has been implemented and merged separately. Also, in #254 (comment), I mentioned that I would welcome a separate white-space normalizing commit. This still holds.
And the latest commit was only removing 5 lines, but there was another extra 16 additions and deletions (probably due to some auto formating of tabs to spaces etc)
Yep. Is this is problem?
from wwwsqldesigner.
Ok, let me file a PR using the auto formatter esbenp.prettier-vscode
, we see if it is suitable. Basically, whitespace additions and deletions require additional effort to parse (read and understand) and ignore, yet it is something that can be eliminated.
Thanks!
from wwwsqldesigner.
Ok, I have done it, removes all white-spaces + auto format, the tools used are also listed.
All I basically did was open, save, close and commit all changes.
from wwwsqldesigner.
Sorry for not replying earlier. I just accepted the PR. Thanks!
from wwwsqldesigner.
Related Issues (20)
- Collaborative version is down HOT 2
- DATATYPES.getElementsByTagName is not a function HOT 1
- Modernize the app HOT 1
- Feature Request: Use a github repo as XML/SQL storage HOT 2
- Error 500 when save to PostGreSql HOT 3
- Get "Internal Server Error" when save php-file on PHP 8 HOT 2
- Public storage
- postgresql constraints all get the same name HOT 2
- Highlight the foreign key connections when selected HOT 22
- Panning on the map has no effect HOT 6
- [SECURITY] POSTing without token HOT 6
- Integrate with an admin panel HOT 3
- State should not be dirty HOT 8
- Generates only MySQL HOT 2
- mouse cursor lose focused when typing in the text fields HOT 2
- save in the server don't work HOT 4
- Support of other sql dbs like apache superset HOT 1
- Python version to automated sql designs from ORMs HOT 3
- Nullable columns
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from wwwsqldesigner.