GitHub actions

[joeapearson]

• Matrix github actions
• Add credo for linting
• Mix format
• Add docker compose for mongo, including replicaset support

I've added support for matrix builds. Currently there is only one combination:

1. Ubuntu latest
2. MongoDB 4.4
3. Elixir 1.12.3
4. OTP 24.0.6

We can add more combinations later.

[scottmessinger]

I'm really excited about this!

For our app, we've been using this to start mongodb:

      - name: Start MongoDB
        uses: supercharge/[email protected]
        with:
          mongodb-version: 4.0.27
          mongodb-replica-set: "replica-set"

I'm wondering if this might help solve the problem where the tests are failing b/c they can't connect to Mongodb.

[joeapearson]

I'm really excited about this!

Thanks I am too! I think it's going to help us a lot to accept changes going forward.

I'm wondering if this might help solve the problem where the tests are failing b/c they can't connect to Mongodb.

That might be the way we have to go.

For now I'm trying to go down the docker-compose approach because then there is a guaranteed identical environment in dev and CI.

I've nearly got it running on my machine (most of the tests pass too!).

It looks like:

1. SSL tests are always disabled in test. Let's fix this one day. I propose not to fix this in this branch and create a separate issue.
2. Docker test are always disabled in test. Similarly, I think this could be fixed in a separate issue but is less important IMO.

I'd like to report test coverage out too but that'll be a separate effort.

[joeapearson]

For our app, we've been using this to start mongodb:

Scott FYI I tried this and it did get further but since the supercharge action only creates a single member replicaset some of the tests that exercise different write concerns won't work.

[ankhers]

A simple first step could be to decide the upper and lower bounds of both Elixir and OTP versions you want to test against and just have the matrix be those two sets. If you want to really make sure everything works, you can install as many versions between those bounds that you want to test against.

[ankhers]

This is fair. I believe there was one in the past but it may be gone now. I think the number of people developing on windows is fairly low as is.

[joeapearson]

A simple first step could be to decide the upper and lower bounds of both Elixir and OTP versions you want to test against

Yes agreed. Again my preference is to start as basic as possible and then gradually expand our "official support" as we get used to taking care of this repo. I started a discussion about that in #372 .

Thanks very much for the unexpected code review by the way, it was my understanding that you'd preferred to put the mongo work down for now otherwise I would have asked you directly.

[joeapearson]

This is fair. I believe there was one in the past but it may be gone now. I think the number of people developing on windows is fairly low as is.

I'm on a mac but when I have had to use Windows it's maddening to find that it's not been properly taken care of. I think this should be a separate issue though.

[ankhers]

Thanks very much for the unexpected code review by the way, it was my understanding that you'd preferred to put the mongo work down for now otherwise I would have asked you directly.

My pleasure. Part of it is I am in the process of modifying the GH actions at $DAYJOB, so I figured I could pitch in a bit here since it is all still fresh in my head. It's not that I am totally putting this project down, it is just that I don't personally use mongo in any of my own projects at the moment, so I may not be the best person to decide direction of the packages. However, I am still more than happy to pitch in where it makes sense. I think that is mostly in code reviews or if you or anyone else has have questions about the code, I am more than happy to try and help out.

[YasirAsar]

I'm so happy to see this repo active after long time. Thanks guys!!!

[scottmessinger]

@joeapearson I just cut a release! https://hex.pm/packages/mongodb/1.0.0-beta.1

The next step is to get this in before we hit 1.0.0.

[etroynov]

@joeapearson

any news?

[scottmessinger]

@etroynov It's shipped!