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

add the platform attributes for 4.6 and 5.0 (for Metacello Preview) #366

Merged
merged 1 commit into from
Aug 24, 2015
Merged

add the platform attributes for 4.6 and 5.0 (for Metacello Preview) #366

merged 1 commit into from
Aug 24, 2015

Conversation

pdebruic
Copy link
Collaborator

issue #365

@dalehenrich dalehenrich changed the title add the platform attributes for 4.6 and 5.0 add the platform attributes for 4.6 and 5.0 (for Metacello Preview) Aug 24, 2015
@dalehenrich
Copy link
Member

It looks like the code for 1.0.0-beta.32.17 is no longer passing - these results do not include your changes ... I suspect tests have changed since then .. so I am seriously considering merging this without passing travis tests ... remind me why I thought that we needed to make these changes:)

@pdebruic
Copy link
Collaborator Author

After merging this change we will get a new SHA we can reference in the ConfigurationOfMetacelloPreview (which we will update then all over the place) to ensure that when someone using Squeak 4.6 or 5.0 runs an #ensureMetacello process they get a version of Metacello that knows about the platform attributes for Squeak 4.6 & 5.0.

I think its fine to merge because it only affects the platform attributes on Squeak, and the code worksforme.....

dalehenrich added a commit that referenced this pull request Aug 24, 2015
add the platform attributes for 4.6 and 5.0 (for Metacello Preview)
@dalehenrich dalehenrich merged commit ffa18f4 into Metacello:issue_365 Aug 24, 2015
@dalehenrich
Copy link
Member

Now I will start work on updating the Configuration ... open an issue for tracking

@pdebruic
Copy link
Collaborator Author

We started the configuration on Friday I think. I've got it on my laptop. I'll push it and make a PR.

@pdebruic pdebruic deleted the squeak-46-50-attributes branch August 25, 2015 00:33
@dalehenrich
Copy link
Member

Whoops, I made the same edits earlier this afternoon: bda06bf ... seeing lots of test failures on both yours and mine - I assume that the problem is that we're hitting github api rate limits running the tests ... I've been restarting my tests and I think they are passing ... so I'm going to nuke your test run and go with my edits since the tests are further along ...

@pdebruic
Copy link
Collaborator Author

Sounds good.

On Aug 24, 2015, at 7:02 PM, Dale Henrichs notifications@github.com wrote:

Whoops, I made the same edits earlier this afternoon: bda06bf ... seeing lots of test failures on both yours and mine - I assume that the problem is that we're hitting github api rate limits running the tests ... I've been restarting my tests and I think they are passing ... so I'm going to nuke your test run and go with my edits since the tests are further along ...


Reply to this email directly or view it on GitHub.

@krono
Copy link
Collaborator

krono commented Aug 28, 2015

patiently waiting :)

@pdebruic
Copy link
Collaborator Author

@krono see the comment on issue #365 ...

@dalehenrich
Copy link
Member

Sorry @krono, I've got a lot of things on my plate right now and if the
tests all ran green it would be much easier for me to just press a merge
button ... I have yet to see 4.6 and/or 5.0 pass on travis either ...

On 8/28/15 7:02 AM, Tobias Pape wrote:

/patiently waiting/ :)


Reply to this email directly or view it on GitHub
https://github.com/dalehenrich/metacello-work/pull/366#issuecomment-135782217.

@krono
Copy link
Collaborator

krono commented Aug 31, 2015

No problem :) I am still on vacation and hence quite unresponsive

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.

3 participants