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.

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

Merge master into soroban soroban-xdr-next #4990

Merged
merged 55 commits into from
Jul 28, 2023

Conversation

2opremio
Copy link
Contributor

I needed this because of #4986 , but a merge was due anyways.

dependabot bot and others added 30 commits February 27, 2023 14:46
Bumps [golang.org/x/text](https://github.com/golang/text) from 0.3.7 to 0.3.8.
- [Release notes](https://github.com/golang/text/releases)
- [Commits](golang/text@v0.3.7...v0.3.8)

---
updated-dependencies:
- dependency-name: golang.org/x/text
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Bump core image to latest stable release v19.10.0
Suppress Core timeout error when ingestion state machine is in build state.
…tellar#4877)

* 4856: Update default for  --captive-core-use-db to true
* Update CHANGELOG.md
…t-tx-ledger

Parse LIMIT_TX_QUEUE_SOURCE_ACCOUNT in core config
Pinning and updating golang and ubuntu images
* tools/goreplay-middleware: Add goreplay middleware
* Fix linter errors

---------
Co-authored-by: Bartek Nowotarski <bartek@nowotarski.info>
* all: Fix improper use of errors.Wrap

`errors.Wrap` method returns nil if the first argument passed is also nil.
If `errors.Wrap` is copied from a condition like `if err != nil` to another
one which  also returns `errors.Wrap` but does not overwrite `err` before
the returned value will always be `nil`.

* Update services/horizon/internal/db2/history/claimable_balances.go

Co-authored-by: George <Shaptic@users.noreply.github.com>

---------

Co-authored-by: George <Shaptic@users.noreply.github.com>
Co-authored-by: Tsachi Herman <24438559+tsachiherman@users.noreply.github.com>
…lang.org/x/text-0.3.8

Bump golang.org/x/text from 0.3.7 to 0.3.8
…er (stellar#4956)

The default value of STELLAR_CORE_URL (localhost:11626) in standalone network mode doesn't work. We need to explictly set STELLAR_CORE_URL to http://host.docker.internal:11626, to allow Horizon to access the host container's port to connect with the core container.
Copy link
Contributor

@tamirms tamirms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@2opremio I left some comments on the merge commit

@2opremio 2opremio force-pushed the merge-master-xdr-next branch from 24a7b37 to 43541de Compare July 28, 2023 12:00
@2opremio
Copy link
Contributor Author

@tamirms I amended and force-pushed the merge. PTAL

@2opremio 2opremio changed the base branch from master to soroban-xdr-next July 28, 2023 12:02
@2opremio
Copy link
Contributor Author

Also, I changed the base branch. It was master and it should had been soroban-xdr-next

@tamirms tamirms self-requested a review July 28, 2023 12:15
Update command_line_args_test.go

Remove command_line_args_test.go
@tsachiherman tsachiherman merged commit 2078669 into stellar:soroban-xdr-next Jul 28, 2023
@2opremio
Copy link
Contributor Author

2opremio commented Jul 28, 2023

Why did you merge? It contains a change I cherry-picked for testing. Let's please let posters merge (unless they say otherwise).

@2opremio 2opremio deleted the merge-master-xdr-next branch July 28, 2023 19:05
@2opremio 2opremio restored the merge-master-xdr-next branch July 28, 2023 19:09
@2opremio
Copy link
Contributor Author

@tsachiherman Reverting, since the commits where squashed instead of merged. Causing a million conflicts when merging master again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.