Skip to content
This repository was archived by the owner on Jun 19, 2020. It is now read-only.

(FACT-2538) Don't save core and legacy facts in collection if they have no value#441

Merged
Filipovici-Andrei merged 6 commits intomasterfrom
FACT-2538
Apr 13, 2020
Merged

(FACT-2538) Don't save core and legacy facts in collection if they have no value#441
Filipovici-Andrei merged 6 commits intomasterfrom
FACT-2538

Conversation

@BogdanIrimie
Copy link
Copy Markdown
Contributor

No description provided.

@BogdanIrimie BogdanIrimie requested review from a team April 9, 2020 14:27
Copy link
Copy Markdown
Contributor

@Filipovici-Andrei Filipovici-Andrei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, at least one unit test would be better😆

@BogdanIrimie
Copy link
Copy Markdown
Contributor Author

Added unit tests.

@BogdanIrimie BogdanIrimie changed the title (FACT-2538) Don't save facts in collection if they have nill value. (FACT-2538) Don't save core and legacy facts in collection if they have no value Apr 9, 2020
@BogdanIrimie BogdanIrimie added the bugfix Something isn't working label Apr 9, 2020
Comment thread lib/models/fact_collection.rb Outdated
Comment thread spec/facter/model/fact_collection_spec.rb Outdated
Comment thread spec/facter/model/fact_collection_spec.rb Outdated
Comment thread spec/facter/model/fact_collection_spec.rb
Comment thread spec/facter/model/fact_collection_spec.rb Outdated
Comment thread spec/facter/model/fact_collection_spec.rb Outdated
@Filipovici-Andrei Filipovici-Andrei merged commit 0954f8a into master Apr 13, 2020
@Filipovici-Andrei Filipovici-Andrei deleted the FACT-2538 branch April 13, 2020 07:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bugfix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants