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

Refactor String url -> URI url #77

Closed
ljleppan opened this issue Sep 4, 2015 · 1 comment
Closed

Refactor String url -> URI url #77

ljleppan opened this issue Sep 4, 2015 · 1 comment
Labels
tmc-legacy-course

Comments

@ljleppan
Copy link
Member

ljleppan commented Sep 4, 2015

Use instances of java.net.URI instead of Strings where ever possible.

See testmycode/tmc-netbeans#115 for related effort in TMC-Netbeans.
This was referenced Sep 4, 2015
Open
Closed
@DarthKipsu
Copy link
Contributor

I'll take this one.

DarthKipsu added a commit to DarthKipsu/tmc-core that referenced this issue Sep 5, 2015
@DarthKipsu
changes all url string to URI type
2019417 
Fixes testmycode#77
@DarthKipsu DarthKipsu mentioned this issue Sep 5, 2015
Merged
@jamo jamo closed this as completed in #79 Sep 11, 2015
wakr pushed a commit to wakr/tmc-core that referenced this issue Sep 12, 2015
@DarthKipsu @wakr
changes all url string to URI type
dbbf735 
Fixes testmycode#77
jamo pushed a commit that referenced this issue Sep 13, 2015
@DarthKipsu @wakr
changes all url string to URI type
2c72761 
Fixes #77

refactor strings to URI changes

arrange two more imports
jamo pushed a commit that referenced this issue Sep 13, 2015
@DarthKipsu @wakr
changes all url string to URI type
335e0a6 
Fixes #77

refactor strings to URI changes

arrange two more imports

fix indentation

fix indentation and import orders

fixed little bug

Brought back zip_url because it got lost after rebasing
tieSari pushed a commit to tieSari/tmc-core that referenced this issue Sep 18, 2015
@DarthKipsu
changes all url string to URI type
e07d49e 
Fixes testmycode#77

refactor strings to URI changes

arrange two more imports
tieSari pushed a commit to tieSari/tmc-core that referenced this issue Sep 18, 2015
@DarthKipsu
changes all url string to URI type
37a8000 
Fixes testmycode#77

refactor strings to URI changes

arrange two more imports

fix indentation

fix indentation and import orders

fixed little bug

Brought back zip_url because it got lost after rebasing
tieSari pushed a commit to tieSari/tmc-core that referenced this issue Sep 18, 2015
@DarthKipsu
changes all url string to URI type
53757c5 
Fixes testmycode#77

refactor strings to URI changes

arrange two more imports
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tmc-legacy-course
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ljleppan @DarthKipsu