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

[WIP] Integrated Modelling using Distributed Repositories #124

Closed
wants to merge 2 commits into from

Conversation

zesaro
Copy link

@zesaro zesaro commented Sep 10, 2018

  • Start and end date: 2018-09-12 to 2019-03-12
  • Contributor: Sascha Zeller, @zesaro
  • Supervisor: Lukas Harzenetter, @lharzenetter

Additional tasks:

  • Rename Variables (Namespace)
  • Check Repository Location
  • Update Frontend (Administration)
  • Initialise on an already existing Repository
  • Support private repositories

saatkamp pushed a commit that referenced this pull request Jan 26, 2019
fix highlighting of 'Other Elements' tab
@zesaro zesaro force-pushed the thesis/multiple-repositories branch from 79cefd7 to 25b3c6f Compare February 6, 2019 10:46
@zesaro zesaro force-pushed the thesis/multiple-repositories branch from 54e42e9 to 14bc2c7 Compare March 2, 2019 13:46
@zesaro zesaro force-pushed the thesis/multiple-repositories branch from 1ab8224 to 21c09a6 Compare March 11, 2019 00:42
@lharzenetter
Copy link
Member

How does the implementation work? Please add documentation on how to use your implementation to the ADR section underneath docs and follow hereby the ADR template.

Copy link
Member

@lharzenetter lharzenetter left a comment

Choose a reason for hiding this comment

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

Please consider the comments and update the documentation as mentioned before.

Copy link
Member

@lharzenetter lharzenetter left a comment

Choose a reason for hiding this comment

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

Could you check the last ToDos? Otherwise, LGTM :)

Copy link
Member

@lharzenetter lharzenetter left a comment

Choose a reason for hiding this comment

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

Does the code now work as expected? If so - please create a single commit as described in the toolchain.

@zesaro
Copy link
Author

zesaro commented Jun 26, 2019

I will try to check it out by the end of the week.

Signed-off-by: Sascha Zeller <zeller.dev@gmail.com>
Signed-off-by: Sascha Zeller <zeller.dev@gmail.com>
@lharzenetter
Copy link
Member

Follow up at eclipse#415

@lharzenetter lharzenetter deleted the thesis/multiple-repositories branch September 16, 2019 06:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

5 participants