Comments (2)
This is for sure just a stylistic decision, but it's one we feel is important. I agree that it's valid syntax, but we tend to find it harder to follow the logic when things are defined later in the file, or at random, so we standardize to define first, whether it's a function expression or declaration, then there's no question as to if it will work, and where to find the logic.
The error message comes from eslint, so while it's possible to change, chances are I'll leave it be for now, since changing it would require a special edge case exception.
I'll leave this open for now, in case you have any thoughts on it, but I'll probably close it soon just to tidy things up.
Let me know though if there are any issues you have with this decision :)
from liferay-frontend-source-formatter.
After reading this it makes sense why that would be confusing pre es6. Thanks for the clarification!
from liferay-frontend-source-formatter.
Related Issues (20)
- The source code did not change at all. Why? HOT 4
- New lines are being added between comments and logic HOT 4
- Integrate with Prettier HOT 13
- SF fails with "try" string and break methods in .js files HOT 2
- Less verbose output HOT 2
- Convert to monorepo? HOT 3
- Add VSCode extension HOT 2
- Force new line on HTML attributes when there are more than two properties (Soy, JSX) HOT 4
- Enforce/disallow HTML void tags' slash HOT 2
- Explanation for "Block level containers need either a class or id" HOT 4
- Wrap JSDoc comments to 80 columns HOT 3
- Running the source formatter breaks some functions calls HOT 2
- Force/disallow new line when chaining HOT 1
- Forcing on code block braces breaks indentation HOT 3
- Force concrete formatting for long conditionals HOT 5
- Force function newline for each parameters HOT 2
- Force newline between different "code blocks" HOT 2
- Provide linting for arrow function parens HOT 1
- Force more consistent JSDoc typing HOT 3
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 liferay-frontend-source-formatter.