X Tutup
Skip to content

Preparation for getting the latest changes for Grease into the master branch --- SeasideSt/Grease 1.3#4

Merged
dalehenrich merged 37 commits intomasterfrom
dev
May 26, 2017
Merged

Preparation for getting the latest changes for Grease into the master branch --- SeasideSt/Grease 1.3#4
dalehenrich merged 37 commits intomasterfrom
dev

Conversation

@dalehenrich
Copy link
Member

Hold off a minute while @marschall figures out if the GRSmalltalkDictionary changes are okay

dalehenrich and others added 30 commits December 21, 2015 15:07
Sync with SmalltalkHub
allow failures for Pharo6.0; .gitignore file; support for GemStone3.4
* added tests for GRDynamicVariable
@jbrichau
Copy link
Member

@dalehenrich @marschall I made a pre-release 1.3.0 on the master branch of SeasideSt/Grease.
I was thinking about sending in a pull request such that GemStone will use latest Grease.

Then I hit this one. Can you update me on the status? All tests are green on the master repo, which is including Gemstone. (https://github.com/SeasideSt/Grease)

I did not follow up on your work at ESUG sprint, but I want to pick up the ball here to release a new Seaside version when we can. Hence, first a new Grease.

Hope you can still remember something :)

@dalehenrich
Copy link
Member Author

I've been using the dev branch of Grease for a very long time ... so I don't have a problem with merging this into the master, but I will wait for @marschall to chime in then we can integrate your changes as well ... @jbrichau inthe mean time, I will pull your changes into the dev branch and see what happens ...

@dalehenrich
Copy link
Member Author

We'll let travis run the tests; @jbrichau the non-mc conflict was in .travis.yml and I chose to use yours

@dalehenrich
Copy link
Member Author

Looking at the timestamp for the original comment in this PR and the timestamp of @marschall second synch (harder than I'd like, but at leat possible) ... the synch comes after the pr was opened, so we did get the latest code from Phillippe :) .... I'll pull the trigger on the merge

@dalehenrich dalehenrich changed the title Preparation for getting the latest changes for Grease into the master branch Preparation for getting the latest changes for Grease into the master branch --- SeasideSt/Grease 1.3 May 25, 2017
@dalehenrich
Copy link
Member Author

@jbrichau I just noticed that 1.3 is marked as pre-release ./... Is that because you are waiting for integration of this PR --- which now includes SeasideSt/Grease 1.3?

@jbrichau
Copy link
Member

@dalehenrich I was unsure if some more commits were going to be necessary before making it a final 1.3.0

Mostly because I was unsure about the state of the Gemstone fork and the comments in here. Also: the smalltalkhub repository does not know about 1.3.0 and the ConfigurationOfGrease does not yet reference this repo.

I think I will start referencing the github repo for Grease and let that one be the master. So, once I committed the ConfigurationOfGrease with a reference to github, I mark it as final release.

Since the GsDevKit / GLASS universe references the GsDevKit/Grease repo, you're welcome to advance master now or after the final release.

@dalehenrich
Copy link
Member Author

I'll wait on the final release to push the dev branch to master, but until then this PR will accumulate the changes that you make on your master branch ... unless something comes up along the way

@jbrichau
Copy link
Member

@dalehenrich I added one more commit on SeasideSt/Grease:master and marked the 1.3.0 as release. I do not see the commit popping up here though...
In any case, I think it's a go.

@dalehenrich
Copy link
Member Author

ah, my bad ... this PR is on the dev branch in this project ... I'll need to create another PR ...

pick up last commit for 1.3
@dalehenrich dalehenrich merged commit 643c2bb into master May 26, 2017
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.

4 participants

X Tutup