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.

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

Runtime attribute class resolution #54

Merged
merged 4 commits into from Jan 3, 2022

Conversation

eikes
Copy link
Collaborator

@eikes eikes commented Dec 17, 2021

No description provided.

@eikes eikes force-pushed the runtime-attribute-class-resolution branch from 0a5d7b0 to 1055003 Compare December 17, 2021 11:54
@eikes eikes force-pushed the runtime-attribute-class-resolution branch from 1055003 to fd3c7d6 Compare December 17, 2021 12:11
lib/ioki/model/base.rb Outdated Show resolved Hide resolved
@eikes eikes force-pushed the runtime-attribute-class-resolution branch 5 times, most recently from 4eaea3c to 41189d2 Compare December 17, 2021 12:26
@eikes eikes force-pushed the runtime-attribute-class-resolution branch from 41189d2 to ddb341c Compare December 17, 2021 13:07
Copy link
Collaborator

@A5308Y A5308Y left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@eikes
Copy link
Collaborator Author

eikes commented Dec 20, 2021

Should I go ahead and merge this or are we waiting for someone or something?

@A5308Y
Copy link
Collaborator

A5308Y commented Jan 3, 2022

@eikes I did already leave for the holidays. You can merge your PRs if you have at least one approval.

@eikes eikes merged commit bed7a7c into master Jan 3, 2022
@eikes eikes deleted the runtime-attribute-class-resolution branch January 3, 2022 10:35
rmehner added a commit that referenced this pull request Jan 6, 2022
This was accidentally introduced in #54 and prevented us from creating
rides, since they don't have a version then as they are just about to be
created.
rmehner added a commit that referenced this pull request Jan 6, 2022
This was accidentally introduced in #54 and prevented us from creating
rides, since they don't have a version then as they are just about to be
created.
rmehner added a commit that referenced this pull request Jan 6, 2022
This was accidentally introduced in #54 and prevented us from creating
rides, since they don't have a version then as they are just about to be
created.
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.

None yet

3 participants