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
hotfix/time harmonizer defend npe #834
Conversation
cc @scherler I have a larger question which is why we are passing this function down in the React context? Couldn't we just export the function from If you agree that approach is reasonable, I am happy to make the changes in this PR. |
🐝 |
@cliffmeyers https://github.com/jenkinsci/blueocean-plugin/blob/master/blueocean-core-js/src/js/config.js in there yeah makes lot of sense |
This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation. |
any change this fixes this one? https://issues.jenkins-ci.org/browse/JENKINS-42318 |
@cliffmeyers looks like this one is worth it (no biggie) |
@michaelneale looking at the code that produces that error message, I don't think it's related to this change. The error this was fixing was mostly coming up in a unit test or storybook environment if I recall. It would probably be better to make the broader fix discussed above once higher priority items get wrapped up. |
@cliffmeyers SGTM |
still seeing harmonizer errors - so this and more is still relevant IMO |
@michaelneale if we merge this now, that's fine, but let's add a ticket to describe the better fix and slap it into the tech debt epic? If you agree w/ that approach, I'll make the ticket later today and the merge this sucker. |
@cliffmeyers we can leave this if you like - just ensure there is a labelled ticket for it. Or merge it, if harmless - really not fussed either way (this doesn't stop most of the errors that I can see, but I don't think it would hurt). |
Closing this - feel free to resurrect one day if important (just too many olde PRs hanging around) |
Description
context.config
- this was breaking storybooks in personalization, and also requires some mocking inPipelineCard-spec
that would be nice to avoid.Submitter checklist
Reviewer checklist
@reviewbybees