Skip to content

Set required fields#42

Merged
arl merged 4 commits intohandle-required-fieldsfrom
set-required-fields
Sep 21, 2020
Merged

Set required fields#42
arl merged 4 commits intohandle-required-fieldsfrom
set-required-fields

Conversation

@arl
Copy link
Copy Markdown
Collaborator

@arl arl commented Sep 17, 2020

❓ What

Tag required fields as such in components configuration

to merge after #41

🔨 How to test

  1. List all steps necessary;
  2. To test this pull request.

✅ Checklists

This section contains a list of checklists for common uses, please delete the checklists that are useless for your current use case (or add another checklist if your use case isn't covered yet).

  • Is there unit/integration test coverage for all new and/or changed functionality added in this PR?
  • Have the changes in this PR been functionally tested?
  • Has make gofmt-write been run on the code?
  • Has make govet been run on the code? Has the code been fixed accordingly to the output?
  • Have the changes been added to the CHANGELOG.md file?
  • Have the steps in CONTRIBUTING.md been followed to update a Go module?

@arl arl requested a review from tommyblue September 17, 2020 16:30
@arl arl force-pushed the handle-required-fields branch 2 times, most recently from 0d1423f to ea9bcca Compare September 18, 2020 16:23
@arl arl force-pushed the set-required-fields branch from ece934d to d3408d0 Compare September 18, 2020 16:24
@arl arl force-pushed the handle-required-fields branch from ea9bcca to f0f69e0 Compare September 21, 2020 08:58
@arl arl force-pushed the set-required-fields branch from d3408d0 to cbc14b7 Compare September 21, 2020 08:59
@arl arl merged commit e1b6265 into handle-required-fields Sep 21, 2020
@arl arl deleted the set-required-fields branch September 21, 2020 11:01
@arl arl restored the set-required-fields branch September 21, 2020 14:25
@arl arl mentioned this pull request Sep 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants