Comments (4)
@baranowb I've implemented the first part of this check (this, but before pushing it, I want to double check with you that the behavior is proper.
On the current payload, BZ1218663 has no component upgrade associated with. This triggers an alert from Bugclerk. On the contrary BZ1218663 has a indeed a component upgrade, and will therefore not trigger the checks...
Sounds good for you ?
from bug-clerk.
OK, right now, there is a drawback, that I think need to be fixed to release those checks as "ERROR" and no just WARNING. Indeed, the current implies that the dependency of any issue needs to be loaded during the initial import or otherwise, from Bugclerk point of view "it does not exist".
So if the filter used to retrieve, for instance, payload tracker issue, does not also load its dependency, the current checks will give back false+ and/or false-.
For now I'm going to release those checks as there are, as warning. Indeed they are usable, if the filter used for the important is well crafted.
from bug-clerk.
Issue #38 has been opened to document the need for enhancement on this front.
from bug-clerk.
Implemented (see commit 9fab31a) and will be released with 0.9.0
from bug-clerk.
Related Issues (20)
- Check Request #38 - Issue attached to payload but not assigned to anybody HOT 1
- Upgrade drools from 6.5 to something better (ie. 7.4.1.Final) HOT 3
- Fix dependency convergence HOT 5
- Refactor mockBug into mockBzIssue for consistency and clarity purpose HOT 1
- Evaluate Helpers - introduce common, object to allow greater detail of error? HOT 3
- Get ride of the AphroditeClient
- Don't null check Optional variable HOT 1
- Drools 7 - does it support more nicely stream and Optional ?
- FixVersionAndSprintMustMatchBetweenSiblings#fixVersionAndSprintNotMatchingBetweenSiblings is wrong HOT 2
- Check Request #39 - Issue state is either MODIFIED or ASSIGNED after PR was closed HOT 1
- Check Request #40 - Issue in on the payload but is missing some acks HOT 1
- HighPriorityIssueNotAssigned should only be triggered for Critical and Blockers, not Major HOT 1
- Migrate unit tests to JUnit5 ?
- Wrong HighPriorityIssueNotAssigned output format. It needs a blank space between the message and level HOT 1
- The bug clerk is mistaken about unassignement HOT 1
- Implementing Checks to tool CVE handling process HOT 1
- Check Request #41 - not allow multiple PRs against different repos. HOT 3
- Message enhancements HOT 2
- Upstream checks should NOT be limited to BZ issue
- IllegalFormatClassException during build HOT 1
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 bug-clerk.