Skip to content

[Fix #302] Make ad work with prismatic/schema #324

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

Merged
merged 1 commit into from
Apr 12, 2016
Merged

Conversation

expez
Copy link
Member

@expez expez commented Apr 7, 2016

No description provided.

@benedekfazekas
Copy link
Member

it seems to me that you submitted this commit in #325 too... I guess that was not the intention?! (see my comments on the code there)

@expez
Copy link
Member Author

expez commented Apr 9, 2016

Actually was intended, as they touched the same code and I tried to reduce the amount of time spent dealing with merge conflicts.

@benedekfazekas
Copy link
Member

I am a bit confused then. Where do you want to track this commit? in this PR or in #325?

@expez
Copy link
Member Author

expez commented Apr 9, 2016

Here, I'll rebase etc if you want changes to this commit

@benedekfazekas
Copy link
Member

strangely I can't find that comment of mine. I was only pointing out that perhaps worth mentioning in the changelog that this understands all prefixed def like things from now on (prismatic is just an example)

@expez expez force-pushed the ad-and-schemas branch 2 times, most recently from 97c838d to 435b125 Compare April 12, 2016 14:11
@expez expez merged commit a488350 into master Apr 12, 2016
@expez expez deleted the ad-and-schemas branch April 12, 2016 14:23
@benedekfazekas
Copy link
Member

nice 👍

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.

2 participants