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

[Enhancement] Fix code style #1837

Merged
merged 63 commits into from
Feb 6, 2025
Merged

[Enhancement] Fix code style #1837

merged 63 commits into from
Feb 6, 2025

Conversation

Tiagospem
Copy link
Contributor

⚡ PowerGrid - Pull Request

  • Bug fix
  • Enhancement
  • New feature
  • Breaking change

Description

Just some code style improvements
...

Related Issue(s):

Documentation

This PR requires Documentation update?

  • Yes
  • No
  • I have already submitted a Documentation pull request.

@Tiagospem
rename datasources to datasource
86a9b17
@Tiagospem
remove unnecessary cast confirm method already returns a boolean
41da80d
@Tiagospem
declare $table name
06ee40d
@Tiagospem
remove redundant static and fix code style
7ad86fa
@Tiagospem
remove duplicated braces
3ee03ed
@Tiagospem
add missing import
403c819
@Tiagospem
add missing import
75024f8
@Tiagospem
add missing import
6519325
@Tiagospem
add missing import
dbc7e14
@Tiagospem
add missing import
e9f0bab
@Tiagospem
add missing import
8d5a3ee
@Tiagospem
add missing import, fix code style and add @throws
2dcd49c
@Tiagospem
rename DataSource as DataSourceCollection and add missing throws
8641985
@Tiagospem
add missing import, fix code style
54bdfb4
@Tiagospem
fix code style
67c5706
@Tiagospem
fix code style and add missing import
45bcf20
@Tiagospem
add missing import
2dbf872
@Tiagospem
add missing import
458c68e
@Tiagospem
add missing import and throws
014a8e6
@Tiagospem
add missing throws
d4b3844
@Tiagospem
remove [] from collect and replace toArray() from all()
88399fa
@Tiagospem
replace !empty by filled and added missing throws
dc7b9b8
@Tiagospem
added missing imports
e4612b7
@Tiagospem
added missing imports
71bf110
@Tiagospem
fix code style
da8b080
@Tiagospem
fix code style
1144532
@Tiagospem
fix code style and added missing imports
967299f
@Tiagospem
fix code style and replace toArray() by all()
bc61c27
@Tiagospem
fix code style and added missing imports
5a1f1ae
@Tiagospem
fix code style and remove unnecessary parameters
4fbca74
@Tiagospem
added missing imports
4636aeb
@Tiagospem
added missing imports
0b7d99d
@Tiagospem
added missing import
17f6168
@Tiagospem
added missing imports
091e6dc
@Tiagospem
added missing imports and replace toArray() by all()
2689a22
@Tiagospem
replace boolval by filled
1b9db23
@Tiagospem
added missing throws and fix ode style
1ec1eba
@Tiagospem
fix code style and added missing import
d7dbd61
@Tiagospem
fix code style and added missing import
dd830cb
@Tiagospem
fix code style
00966cb
@Tiagospem
fix code style and added missing import
b7a9a3e
@Tiagospem
fix code style
2404902
@Tiagospem
added missing import
44e37a2
@Tiagospem
added missing throws
bc227b6
@Tiagospem
added missing import and throws
f4ad1aa
@Tiagospem
added missing import
2052b94
@Tiagospem
added missing import and throws
9478822
@Tiagospem
fix code style
e94e898
@Tiagospem
replace by filled
fff4d24
@Tiagospem
added missing import
0595387
@Tiagospem
added missing import
af2c53e
@Tiagospem
fix code style
29697b2
@Tiagospem
added missing import
51ef242
@Tiagospem
added missing import
606c794
@Tiagospem
fix code style
b515555
@Tiagospem
rollback changes
2a2b9a4
@Tiagospem
use only blank
99bfd03
@Tiagospem Tiagospem changed the title Enhancement/fix code style [Enhancement] Fix code style Feb 2, 2025
@luanfreitasdev luanfreitasdev merged commit 7e6682a into Power-Components:6.x Feb 6, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@luanfreitasdev luanfreitasdev luanfreitasdev approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Tiagospem @luanfreitasdev