Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Nov 28 23:39
    preardon commented #1855
  • Nov 28 23:26
    iancooper commented #1855
  • Nov 28 22:53
    preardon commented #1855
  • Nov 28 21:56
    iancooper commented #1855
  • Nov 28 18:53
    iancooper commented #1855
  • Nov 28 17:17
    preardon commented #1855
  • Nov 26 09:08
    zhchlmm starred BrighterCommand/Brighter
  • Nov 24 17:21
    vinhnxv starred BrighterCommand/Brighter
  • Nov 24 10:06
    StevenTCramer starred BrighterCommand/Brighter
  • Nov 23 20:28
    adersonrangel starred BrighterCommand/Brighter
  • Nov 23 12:33
    dotlive starred BrighterCommand/Brighter
  • Nov 23 12:05

    holytshirt on 9.0.1-preview.5

    (compare)

  • Nov 23 11:50

    holytshirt on master

    Ensured a new CommandProcessorP… (compare)

  • Nov 23 11:50
    holytshirt closed #1866
  • Nov 23 11:32
    preardon assigned #1867
  • Nov 23 11:32
    preardon opened #1867
  • Nov 23 10:48
    preardon synchronize #1866
  • Nov 23 10:02
    holytshirt commented #1857
  • Nov 22 22:50
    preardon synchronize #1866
  • Nov 22 22:16

    dependabot[bot] on nuget

    (compare)

Ian Cooper
@iancooper
Apologies, the EventStore job was a mis-merge by me (not sure when). It was supposed to be on a branch. As it only fails one job, and I still need to fix it, along with Kakfa and AWS, just ignore over my making it worse with a rollback. I am suspicious the event store issue is the health check
Ian Cooper
@iancooper
OK, the EventStore job is fixed. On to AWS as GitHub Actions has some support that ought to allow us to run our SQS tests directly on AWS
Ian Cooper
@iancooper
@/all BTW I was demoing Brighter at JE, and I think our sample code needs a little bit of a clean up, and to lean more heavily on HostBuilder
It will take me a while to grab that, but it's a great first issue if anyone was looking to contribute
Ian Cooper
@iancooper
@/all Darker now has a basic GitHub Actions build. I'll fix it up into separate jobs, like Brighter, so that it is easier to see without having to open the build up, what steps an and passed.
Ian Cooper
@iancooper
I killed the AppVeyor build for Darker as it was failing due to SDK versions.
I'll add a Nuget feed etc to Darker too
I need to fix up a Release build we can trigger for NuGet deployment for both
Ian Cooper
@iancooper
I've moved Brightside over to GitHub Actions as well.
Again it's just a basic build for now, because we don't have the same test separation we have with Brighter, I think it would be nice to achieve that, as the feedback is far better. It doesn't deploy the package to PyPy as yet. I need release builds that deploy across the board
But everything is in GitHub Actions now
Ian Cooper
@iancooper
OK, so the Kafka build has been fixed. A combo of issues with the GitHub Actions services (don't have a blank line between them folks), Kafka 1.5+ no longer defaulting to auto-create topics, and the lack of our usual publisher/creator classes for mapping our message type onto the native type. All of those are fixed. I dropped all apart from one test, as badly written and have some fixes for Kafka to go in over the next couple of days. So this is really just a confidence test. Some of those 'fixes' will be useful to complete the Redis Streams work, which has similarities to Kafka in some requirements. For example, we currently use Null for the partition key value on Kafka. We probably don't want random assignment, but an actual well identified 'partition key' on our message type, that is a string type. We also want to change how we handle getting and committing the offset etc. So expect some more changes here.
Ian Cooper
@iancooper
@holytshirt I am thinking of making it easy to push to NuGet using the approach described here: tag a version, run a push to NuGet. This would be a separate worfklow to the CI build, because you trigger it with a version tag:
I worry about stored value as we don't drop releases very fast, and I would like to up our cadence to at least once a quarter. I'm not sure that a framework should release a lot faster than that.
@holytshirt I know you had some release issues, to do with the changes to the schema of the Db. Can you remember what those were? I'd like to fix if we can, and start releasing on a regular schedule.
Toby Henderson
@holytshirt
@iancooper we need to get a beta of v9 out ...
@iancooper if your build is always creating versioned packages and adding to them to the github package feed, you can just choose a package to promote to nuget?
Toby Henderson
@holytshirt
@iancooper the database stuff is some headers are missing when written to the database and then read back from the database. Think is correlationId is one
@iancooper Also No Outbox doesn't work
Ian Cooper
@iancooper
@holytshirt I don't know if I can promote a NuGet package from the GitHub feed. I was assuming that 'good enough' was get a green build, test, and then trigger a build by tagging. There is complexity about build we test vs. build we tag there, I guess it's a question of do we think it's a real risk
Right, are there issues for those two Outbox issues, then I can tackle them as a priority.
I can raise them, but if you have detail that would help
Toby Henderson
@holytshirt
@iancooper ello Coops, I finally have some time again. Going to catch up on what has been happening and create those issues
Ian Cooper
@iancooper
@holytshirt . Let everyone here know if you get the GitHub discussions happening
@holytshirt PS tempted to adopt this. Thoughts? https://www.conventionalcommits.org/en/v1.0.0/
Toby Henderson
@holytshirt
@/all I've turned on Discussions tab on Brighter repo, so probably the best place to do Q&A, we'll see how it goes.
@iancooper I've created a project for finishing and releasing V9 https://github.com/BrighterCommand/Brighter/projects/5
Ian Cooper
@iancooper
Cool, will talk about Conventional Commits there
Toby Henderson
@holytshirt
@iancooper it is tempting, can see a reason for not using Conventional Commits
Ian Cooper
@iancooper
@/all We are busy getting V9 out of the door. We recommend using the new Discussions that we have enabled for the project if you want to talk Brighter. We'll try to keep an eye out here, but the comments should ping us and will be better