search

ctrl-Felix / cosmospy-protobuf

Cosmos Protobuf library
16 stars 14 forks source link

Update for v2 Sentinel Upgrade #14

Closed MathNodes closed 6 months ago

MathNodes commented 11 months ago

Also, there were variables named from in some of the protobuf files and I had to rename them to frm since from is a python keyword.

Please merge. ty

ctrl-Felix commented 11 months ago

Why are there two sentinel config files? Also is there a way for you to exclude the worflow tg notify please?

MathNodes commented 11 months ago

I just got back from vacation where there was no cell phone service, internet or TV. It was marvelous.

Anyhow, I had two sentinel config files because one was for current and the other for the development. We are using development branch now as we prepare for the upgrade.

I will be making a git push to update this library per your request to exclude the workflow and contain only one sentinel config file. I also noticed there was a namespace collision in v2 sentinel protobuf file that I had to fix (gogoproto)

Will push an update later this week.

MathNodes commented 11 months ago

Ok. I pushed it tonight. Check and verify.

ctrl-Felix commented 11 months ago

Can you check on the conflicts please? Actually, do you see them or is it only me who sees them?

MathNodes commented 10 months ago

All set. I removed the conflicts.