Add base definition of properties

[fredericbarthelet]

Closes: #8017
Addresses: #8016

[codecov-commenter]

Codecov Report

Merging #8222 into master will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #8222      +/-   ##
==========================================
- Coverage   88.10%   88.04%   -0.06%     
==========================================
  Files         248      248              
  Lines        9384     9329      -55     
==========================================
- Hits         8268     8214      -54     
+ Misses       1116     1115       -1     

Impacted Files	Coverage Δ
lib/configSchema.js	100.00% <ø> (ø)
lib/plugins/aws/provider/awsProvider.js	93.11% <ø> (ø)
lib/classes/ConfigSchemaHandler/index.js	89.52% <100.00%> (ø)
lib/plugins/aws/package/compile/functions/index.js	97.03% <100.00%> (-0.22%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3e9e6aa...3a30bbe. Read the comment docs.

[medikoo]

@fredericbarthelet is this intentionally posted as draft pull request?

[fredericbarthelet]

@medikoo yes it is intentional, I'm not quite finished yet on this one :) Should be completed by the end of the day

[fredericbarthelet]

@medikoo this PR is now ready for review when you can :)

[medikoo]

@fredericbarthelet great thanks for that!

In master I've reordered definitions so alphabetical order is respected, and I see now this branch conflicts due to that. Can you rebase it against master?

[fredericbarthelet]

@medikoo I sorted the properties :) Let me know what you think

[medikoo]

@fredericbarthelet that looks outstanding! Great thanks for that. Please see my remarks, and let me know what you think

[fredericbarthelet]

@medikoo I took into account all of your comments. Could you have another look at my PR and let me know if something is still missing ? thanks :)

[medikoo]

Thanks @fredericbarthelet it looks very good! We're nearly there, please see my comments

[fredericbarthelet]

@medikoo updated my PR with your latest comments :)

[medikoo]

@fredericbarthelet sorry for confusion. Tests exposed that actually package.individually is supported on function level (my bad - I totally forgot it's the case).

In light of that, we need to recognize that option on function level

[fredericbarthelet]

@medikoo no problem, updated it with allowed individually property at function level :)

[medikoo]

@fredericbarthelet that looks great!

I have just last few minor (totally nit picking) style suggestions, so we're noise free and consistent. Let me know

[fredericbarthelet]

@medikoo no problem, updated to match alphabetical order and removed unecessary awsKmsArnString definition

[medikoo]

Thank you @fredericbarthelet !