@filipesperandio

Seems like I don't have the enough permissions for requesting your review, that's why I ping you here.

Also, you will noticed the I have changed the base docker imagealpine:edge to an specific version. While using thealpine:edge image I suffered from different dns resolutions problems which were not allowing a successfulcompose install, it wasn't able to fetch certain packages. After hours of checking if those packages existed and other potential explanations I ended up changing the base image and it started to work. I didn't perform any further research on that due to the fact that we are a little bit tight regarding schedule.

I addressed as CC issues as I could, there are some remaining that have been here for a long time, so I think we could temporary ignored them.

One more thing, my idea is to create abeta channel for this plugin so I think it would be nice to merge this work against abeta branch. Could you create that branch please ? 🙏🏼

One more other thing 😆 , I think we could include this workhttps://github.com/codeclimate/codeclimate-phpcodesniffer/pull/80/files into the release of thebeta channel. WDYT ?, after merging this one I can rebase that one, test it and update that pr to point tobeta. And do you have an idea of whyci/circleci is being displayed as pending 🤔 ?

Seems like I don't have the enough permissions for requesting your review, that's why I ping you here.

Permissions granted, no need the fork either anymore.

Also, you will noticed the I have changed the base docker image alpine:edge to an specific version. While using the alpine:edge image I suffered from different dns resolutions problems which were not allowing a successful compose install, it wasn't able to fetch certain packages. After hours of checking if those packages existed and other potential explanations I ended up changing the base image and it started to work. I didn't perform any further research on that due to the fact that we are a little bit tight regarding schedule.

👍

will look at Circle CI now.

I think the circle issue is due to the fork. we don't allow it to pass in credentials to forked PRs.
Now that you have permissions for the repo, can we try pushing a non-forked branch and update the PR? (I don't recall doing it, not sure we can or a new PR is needed)

Seems like we can only change base branch... so maybe a new one?

beta created

#80 merged intobeta

@filipesperandio done ! 💪🏼

Replaced by#83