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

Create middleware architecture to add "flag/argument providers" #173

Closed
wesen opened this issue Feb 27, 2023 · 1 comment · Fixed by #181
Closed

Create middleware architecture to add "flag/argument providers" #173

wesen opened this issue Feb 27, 2023 · 1 comment · Fixed by #181
Labels
enhancement New feature or request

Comments

@wesen
Copy link
Collaborator

wesen commented Feb 27, 2023

As more and more ways to set command flags will be added (we already have:

  • CLI
  • viper
  • GET
  • form POST
    ), we should maybe devise some kind of middleware structure to set them, but also transform and log them in some generic fashion.
@wesen wesen added the enhancement New feature or request label Feb 27, 2023
@wesen wesen mentioned this issue Feb 27, 2023
Open
@wesen
Copy link
Collaborator Author

wesen commented Feb 27, 2023

This might also be related to #167

@wesen wesen mentioned this issue Feb 27, 2023
Merged
This was referenced Feb 27, 2023
Merged
Merged
wesen added a commit to wesen/glazed that referenced this issue Feb 28, 2023
@wesen
🎨 Little tweaks to finish up go-go-golems#173 and also go-go-golems#136
b5d7275
@wesen wesen mentioned this issue Feb 28, 2023
Merged
wesen added a commit that referenced this issue Feb 28, 2023
@wesen
Merge pull request #182 from wesen/misc/173/polish-up-173
d239fff 
Polish up #173 and #167
@wesen wesen mentioned this issue Mar 1, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactor parameter layer parsing to allow for prefixes wesen/glazed
1 participant
@wesen