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 library entry point #212

Merged
merged 1 commit into from
Mar 24, 2015
Merged

Conversation

phlipper
Copy link
Contributor

#205 introduced a regression when the entry point file was renamed: lib/ruby-saml.rb → lib/onelogin/ruby-saml.rb. This is causing problems with bundler, and also when integrating with other libraries such as omniauth-saml.

* SAML-Toolkits#205 introduced a regression when the entry point file was
  renamed: `lib/ruby-saml.rb → lib/onelogin/ruby-saml.rb`.
  this is causing problems with `bundler`, and also when
  integrating with other libraries such as `omniauth-saml`.
@pitbulk
Copy link
Collaborator

pitbulk commented Mar 24, 2015

Already reported that and I was waiting for validation, since you already noted that I will merge.
👍

pitbulk added a commit that referenced this pull request Mar 24, 2015
@pitbulk pitbulk merged commit c850f4c into SAML-Toolkits:master Mar 24, 2015
@Lordnibbler
Copy link
Contributor

@pitbulk this PR failed travis FYI, should ALWAYS wait until we're sure we're not merging a breaking change!

thanks for this fix @phlipper

@pitbulk
Copy link
Collaborator

pitbulk commented Mar 24, 2015

@Lordnibbler, sorry, but It seems a Travis issue, tested it on local and worked.

@Umofomia Umofomia mentioned this pull request Apr 1, 2015
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