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

server_variable_list: smarter? #30

Closed
jonthegeek opened this issue Sep 2, 2023 · 1 comment · Fixed by #35
Closed

server_variable_list: smarter? #30

jonthegeek opened this issue Sep 2, 2023 · 1 comment · Fixed by #35
Milestone
0.1: through Auth

Comments

@jonthegeek
Copy link
Owner

Can server_variable_list figure out what to do with a list, and just ingest an apid_list without being told that's what it is?

And from there, can everything?
@jonthegeek jonthegeek added this to the 0.1 milestone Sep 5, 2023
@jonthegeek
Copy link
Owner Author

I realized as_<class>() makes more sense for what I was doing, so this will be closed out largely for "free".

To finish this: Make sure this object makes sense.

jonthegeek added a commit that referenced this issue Sep 8, 2023
@jonthegeek
Standardize server_variable_list() to variables()
3eb2c12 
Closes #30.

This is the only place that variables() appears in the definition, and there's really no reason that it would have to be server-specific anyway.
@jonthegeek jonthegeek mentioned this issue Sep 8, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
{rapid} releases
Archived in project
Milestone
0.1: through Auth
Development

Successfully merging a pull request may close this issue.

Smarter jonthegeek/rapid
1 participant
@jonthegeek