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

Don't use deprecated methods #472

Open
wants to merge 11 commits into
base: dev
Choose a base branch
from

Conversation

StephanEggermont
Copy link
Contributor

No description provided.

macta and others added 9 commits February 12, 2019 22:42
Add links to further documentation to help people learn more 
Also indicate that Calypso is now live in Pharo 7
Update README.md to provide links to more documenation
New release with bug fixes and class parser preparation
Fix test for class comments query which fails in Pharo 8
@dionisiydk
Copy link
Collaborator

There are more places (I count three).
And I would defer this change for later to avoid incompatibility with Pharo 7 for some time.

@dionisiydk dionisiydk changed the base branch from master to dev March 24, 2019 15:24
@dionisiydk
Copy link
Collaborator

Can you split comment part into another PR. So we can update it faster?

And please consider to contribute based on dev branch instead of master.

@StephanEggermont
Copy link
Contributor Author

Can't be delayed for Pharo 8, CI is red

@dionisiydk
Copy link
Collaborator

What CI is red?

Calypso CI is green.
In latest Pharo 8 there was single failed test which is now covered by new version (PR for it in pharo repo)

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