Skip to content

Code reorganization - #1

Merged
slide merged 4 commits into
IronLanguages:masterfrom
slide:code_reorg
May 23, 2017
Merged

slide merged 4 commits into
IronLanguages:masterfrom
slide:code_reorg

Conversation

@slide

@slide slide commented May 21, 2017

Copy link
Copy Markdown
Contributor

There are some test failures that don't occur in "main" and not all of the same tests are being run yet.

@slozier

slozier commented May 21, 2017

Copy link
Copy Markdown
Contributor

Looks like a typo in assert_util.py which is causing most of the failures? I didn't have time to test with the typo fixed.

I wonder if we should wait until after the next release to see how much of an outcry there is before we start ripping out all the silverlight and net35 stuff. Although I guess it's fine to remove it from testing since we're not running those tests anyway...

@slozier

slozier commented May 23, 2017

Copy link
Copy Markdown
Contributor

Maybe move the test stuff into a separate PR?

@slide

slide commented May 23, 2017

Copy link
Copy Markdown
Contributor Author

Ok, I can do that, my reason for putting it here was because I didn't want to fix the tests in the current state, I wanted to move them to the new format while fixing them.

@slozier

slozier commented May 23, 2017

Copy link
Copy Markdown
Contributor

Only reason I'm suggesting using a separate PR is to avoid having the changes getting lost in the noise (we'll need to port them over to ipy3). Once the testing changes are split off we can probably merge in the reorg as is.

@slide

slide commented May 23, 2017 via email

Copy link
Copy Markdown
Contributor Author

@slide slide changed the title [WIP] Code reorganization Code reorganization May 23, 2017
@slide
slide merged commit c5e17d5 into IronLanguages:master May 23, 2017
@slide
slide deleted the code_reorg branch May 23, 2017 19:38
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