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.

Sign up for GitHub

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

Jump to bottom

Automated Fixes Generated By StyleCI #80

Closed
wants to merge 1 commit into from
Closed

Automated Fixes Generated By StyleCI #80

wants to merge 1 commit into from

Conversation

GrahamCampbell
Copy link
Contributor

@GrahamCampbell GrahamCampbell mentioned this pull request Dec 20, 2014
Merged
@jbrooksuk
Copy link
Member

I wonder why this can't merge?

@jbrooksuk
Copy link
Member

Ah, it's to do with the changes I made just before you. Could you run again?

@GrahamCampbell
Copy link
Contributor Author

Yeh.

@GrahamCampbell GrahamCampbell deleted the cs branch December 20, 2014 21:10
@GrahamCampbell
Copy link
Contributor Author

I don't like the namespace change. Are you not going to rename the folder to correct the case first?

@jbrooksuk
Copy link
Member

Yeah, I'm just trying to get it to work, Git is throwing a paddy at me, even with the -f param.

@GrahamCampbell
Copy link
Contributor Author

Just rename it to something wrong, commit, then change it again. I sometimes do that. :)

@jbrooksuk
Copy link
Member

That'll work - I was trying to be clever :P

@jbrooksuk
Copy link
Member

Done :)

@GrahamCampbell
Copy link
Contributor Author

You know, styleci can be enabled for this repo if you want.

@GrahamCampbell
Copy link
Contributor Author

All you need to do is setup a hook.

@jbrooksuk
Copy link
Member

That'd be good, we don't have anything enforcing the style yet.

@jbrooksuk
Copy link
Member

BTW, just pushed a change for composer.json.

@GrahamCampbell GrahamCampbell modified the milestone: V0.1.0 Alpha Jul 25, 2015
@TheoBearman TheoBearman mentioned this pull request Feb 1, 2016
Closed
@karthik-22feb karthik-22feb mentioned this pull request Aug 23, 2016
Closed
@bwalker-telstra bwalker-telstra mentioned this pull request Jun 7, 2017
Closed
@raffus raffus mentioned this pull request Mar 22, 2018
Closed
@danielmclements danielmclements mentioned this pull request Mar 29, 2018
Closed
This was referenced Jun 22, 2018
Closed
Closed
@LordCheta LordCheta mentioned this pull request Jul 4, 2018
Closed
@melvinlemoine melvinlemoine mentioned this pull request Nov 23, 2018
Closed
@tychovbh tychovbh mentioned this pull request May 27, 2019
Closed
@astrakid astrakid mentioned this pull request Feb 17, 2020
Closed
@joschiservice joschiservice mentioned this pull request May 29, 2020
Closed
@DarkShoro DarkShoro mentioned this pull request Nov 27, 2020
Closed
@astrakid astrakid mentioned this pull request Apr 27, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
V0.1.0
Development

Successfully merging this pull request may close these issues.
2 participants
@GrahamCampbell @jbrooksuk