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.

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

Fix various warnings introduced to master #106

Open
TheGuardianWolf opened this issue Mar 15, 2018 · 2 comments
Open

Fix various warnings introduced to master #106

TheGuardianWolf opened this issue Mar 15, 2018 · 2 comments

Comments

@TheGuardianWolf
Copy link
Contributor

TheGuardianWolf commented Mar 15, 2018

Recent changes to master has caused tools to generate warnings about various files. This should be taken care of as a maintenance task.

Each problem is listed below by file.

Size is very small.

WARNING in ./src/main/webapp/app/entities/study/study.route.ts
[11, 83]: trailing whitespace
[55, 15]: " should be '
[58, 27]: " should be '
[59, 24]: " should be '
[63, 17]: " should be '

pom.xml

[WARNING] Some problems were encountered while building the effective model for org.project36.qualopt:qual-opt:war:0.0.1-SNAPSHOT
[WARNING] 'dependencies.dependency.(groupId:artifactId:type:classifier)' must be unique: de.saly:javamail-mock2-fullmock:jar -> duplicate declaration of version 0.5-beta4 @ line 453, column 21

package.json

warning From Yarn 1.0 onwards, scripts don't require "--" for options to be forwarded. In a future version, any explicit "--" will be forwarded as-is to the scripts.

Update: Changed title to reflect larger task at hand.

@softeng-701
Copy link
Collaborator

@TheGuardianWolf APPROVED!

@TheGuardianWolf
Copy link
Contributor Author

This issue also includes resolution of the above closed issue #94

@TheGuardianWolf TheGuardianWolf changed the title TSLint producing linter warning on file Fix various warnings introduced in master Mar 27, 2018
@TheGuardianWolf TheGuardianWolf changed the title Fix various warnings introduced in master Fix various warnings introduced to master Mar 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants