Comments (2)
I don't believe this issue should be targeted at new developers. Instead, I believe it should be requirement of ALL developers. This should be thought of as technical debt. If a developer touches an existing file, they should be responsible for bringing it up to the standard. That way over time we will pay down the debt and have all source up to the standard.
There are also things we could to do to target the CI/CD, such as code reformatting and checking for PRs to ensure the coding standards are met.
from cve-services.
Thanks @david-waltermire-nist. The idea of it being a good first issue at the time was that someone could look over the repo, fix linter errors, and be able to make a low risk pull request to the project. Similar to fixing typos in documentation, it can be a great way to get someone started in a project. I do understand how technical debt works and how a style standard should be obeyed by a dev team. I'll close this issue, as a CI/CD pipeline is coming which will contain a linter that applies the style standard we have chosen to branches and PRs.
from cve-services.
Related Issues (20)
- Implement ADP Role
- ADP User Functionality
- Create ADP org middleware
- Remove ADP container in Reject endpoints
- Create ADP container endpoint HOT 1
- Create and Update APD Container endpoint HOT 4
- Ensure new Organizations with only APD role have ID_quota of 0
- Integration Test Additions and Improvements
- openapi.json is unsuitable for generating bindings HOT 1
- Get records filter by CNA container modified
- Get records filter by specified ADP container modified
- Change GET `/cve-id/:id ` to only return Cve-ids own by the requesting Org
- Create integration test to check id_quota when removing CNA role from Org
- Update ADP container create/update response to specify owning org
- Create endpoint to return CVE-IDs for CVE records the calling org has added ADP containers to
- Publishing a CVE record should throw a 400 if `datePublic` is a future date
- Create and improve tests for cve.controller.js
- Create and improve tests for cve-id.controller.js
- Create and improve tests for org.controller.js
- Create tests for user.controller.js
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 cve-services.