Skip to content

Conversation

@hflynn
Copy link
Contributor

@hflynn hflynn commented Feb 19, 2013

This is PR #254 rebased to develop

Helen Flynn and others added 7 commits February 19, 2013 10:09
Conflicts:
	developers/Clients/ImportLibrary.txt
	developers/Model.txt
Based on the notes I took while discussing
the build and the server code in Dundee.

 * Add model-generation.png from the Nature paper
 * Merged TypeLanguage into Model
 * Added specifics comments about generated files

Conflicts:
	developers/Model.txt
	developers/Server/ExtendingOmero.txt
@hflynn
Copy link
Contributor Author

hflynn commented Feb 19, 2013

@joshmoore This was breaking the docs because your DSL changes weren't on the develop branch but my dev_4_4 changes which I was rebasing were built on top of them, so I've cherry-picked your commit to fix it. Hope that is okay.

@joshmoore
Copy link
Member

@hflynn: works for the moment. Next time just give a shout and I'll rebase. Sorry for having forgotten!

@hflynn
Copy link
Contributor Author

hflynn commented Feb 19, 2013

It's okay, by the time I'd figured out what the issue was it seemed as well to fix it myself. I don't know if there were any other commits on that PR you need to rebase though - I did this one because the image it added being missing was what broke the build.

@joshmoore
Copy link
Member

My PR had 2 other commits. Do you want to try cherry-picking those as well, and then we'll know for sure?

@joshmoore
Copy link
Member

Conflicts:
	developers/Model.txt
The changes are quite small but to maintain line
lengths I formatted the paragraphs.

Conflicts:
	developers/Model.txt
Copy link
Member

Choose a reason for hiding this comment

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

The final dot had been removed in 0fa2d57. While not doing it makes sense, this may cause conflicts in the future.

@sbesson
Copy link
Member

sbesson commented Feb 20, 2013

Apart from very minor comment above, the diffs are the same for the conflicting developer/Model.txt file and the build passed. Ready to merge.

@hflynn
Copy link
Contributor Author

hflynn commented Feb 21, 2013

Have even fixed the tiny nitpick now - ready to merge.

@joshmoore
Copy link
Member

+:beer:

joshmoore added a commit that referenced this pull request Feb 21, 2013
@joshmoore joshmoore merged commit cd5cbf8 into ome:develop Feb 21, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants