Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Config v3 framework integration #383

Merged
merged 26 commits into from Apr 1, 2024
Merged

Config v3 framework integration #383

merged 26 commits into from Apr 1, 2024

Conversation

Blacksmoke16
Copy link
Member

@Blacksmoke16 Blacksmoke16 commented Apr 1, 2024 •
edited

Follow up to #337 to revamp how components are integrated into the framework.

  • Update docs to reference new ATH.configure approach to configuration
    • Drop DI param docs
    • Document the new methods/types added in last PR
  • Make the special member markup not include any docs if not included vs NONE
  • Add specs for framework bundle
  • Rename listeners to remove the _listener suffix
  • Breaking Integrate framework features via new bundle system
    • Refactor format negotiator setup to make use of request matchers
    • Automatically determine framework.debug param value
    • Drop ATH::Parameters type
    • Provides more customization options than before
  • Make ART::Router#strict_requirements nilable to allow for third option

@Blacksmoke16
Re-enable a CORS listener validation spec
09e5914
@Blacksmoke16
Remove focus
4d7d107
@Blacksmoke16
Wire up CORS listener via config
5fc7874
@Blacksmoke16
Move framework.debug param inside of ATH::Bundle
49de629 
Make `CORS::Config` internal and `CORS` listener constructor protected
Start of bundle specs
Rename listener types to remove `_listener` suffix
@Blacksmoke16
Remove property? schema macro
e592eb0 
Improve how property docs are rendered
Support validation of nilable array types
Play around with how to document schema properties via format handler rules
@Blacksmoke16
Add spec for named tuple array types
f6e898e
@Blacksmoke16
Merge master & resolve conflicts
4cf04af
@Blacksmoke16
Expose schema property information to mkdos via JSON in const
ad54a9e 
Create special markup allowing documenting of sub-object properties
@Blacksmoke16
Cleanup & add specs to extension macros
be3a4ee
@Blacksmoke16
Merge master & resolve conflict
803d19d
@Blacksmoke16
Start revamping configuration docs based on new approach
2bd5ff2
@Blacksmoke16
Define additional bundle configuration properties
e12e71b
@Blacksmoke16
Merge remote-tracking branch 'origin/master' into config-v3-framework…
d2b2a44 
…-integration
@Blacksmoke16
Handle default symbol enum property values
f790600 
Properly extract array type from nilable union
@Blacksmoke16
Merge master & resolve conflicts
ee19f4a
@Blacksmoke16
Merge master & resolve conflicts
473c4bd
@Blacksmoke16
Merge master & resolve conflicts
be80a64
@Blacksmoke16
Document ADI extension related types/macros
8d470c5 
Default member docs to an empty string
@Blacksmoke16
Wire up more services via bundle pattern
4217a34
@Blacksmoke16
Merge master & resolve conflicts
34a6d4b
@Blacksmoke16
Merge remote-tracking branch 'origin/master' into config-v3-framework…
1b4712d 
…-integration
@Blacksmoke16
Simplify manually registered service defs
3982128
@Blacksmoke16
Merge remote-tracking branch 'origin/master' into config-v3-framework…
2b643f4 
…-integration
@Blacksmoke16
Refactor format negotiator integration/logic
298dcb6
@Blacksmoke16 Blacksmoke16 mentioned this pull request Apr 1, 2024
Merged
4 tasks
@Blacksmoke16 Blacksmoke16 marked this pull request as ready for review April 1, 2024 03:53
@Blacksmoke16 Blacksmoke16 merged commit edd92dc into master Apr 1, 2024
10 checks passed
@Blacksmoke16 Blacksmoke16 deleted the config-v3-framework-integration branch April 1, 2024 04:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
breaking component:dependency-injection component:framework component:routing kind:documentation kind:feature Brand new functionality
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@Blacksmoke16