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

Rename CypressDev to CypressOnRails #19

Merged
merged 4 commits into from
Jun 20, 2019

Conversation

grantspeelman
Copy link
Collaborator

as mentioned in #17 and #11

@grantspeelman grantspeelman self-assigned this Jun 18, 2019
@grantspeelman
Copy link
Collaborator Author

@justin808 can you enable travis-ci for the project please

Copy link
Contributor

@yagudaev yagudaev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great :)

@@ -202,7 +202,7 @@ add the following file to cypress

```js
// test/cypress/support/on-rails.js
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should the file itself be called cypress-on-rails?

s.author = ["miceportal team", 'Grant Petersen-Speelman']
s.email = ["[email protected]", '[email protected]']
s.homepage = "http://github.com/grantspeelman/cypress-on-rails"
s.summary = "Integrates cypress with rails or rack applications"
s.description = "Integrates cypress with rails or rack applications"
s.post_install_message = 'The CypressDev constant is being deprecated and will be completely removed and replaced with CypressOnRails'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice!

@grantspeelman
Copy link
Collaborator Author

Screen Shot 2019-06-21 at 8 08 59 am

Installed the branch on our project, and post message comes up 👍

@grantspeelman
Copy link
Collaborator Author

Tested the branch on our CI with only updating the gem and it works without any additional changes.

@grantspeelman grantspeelman merged commit 49f1f7b into master Jun 20, 2019
@grantspeelman grantspeelman deleted the rename-to-cypress-on-rails branch June 20, 2019 22:30
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