Skip to content
This repository was archived by the owner on Mar 19, 2024. It is now read-only.

Build fixes from Feature/msz 0706f#17

Draft
miklosszegedi wants to merge 6 commits into
onflow:masterfrom
GetElastech:feature/msz-0706f
Draft

Build fixes from Feature/msz 0706f#17
miklosszegedi wants to merge 6 commits into
onflow:masterfrom
GetElastech:feature/msz-0706f

Conversation

@miklosszegedi

Copy link
Copy Markdown

Goal of this PR

Fixes #

Checklist

Does the PR require tests to be added or updated?

This is the case if this PR updates currently unit tested code with new/removed paths, or if it adds new components
that should be tested.

Ideally PRs should include the tests to code they add, but if for some reason they do not, there should be a TODO
comment in the place where the test should be, linking to a GitHub issue that details what test should be written.

Does the PR require documentation to be added or updated?

This is the case if this PR changes any of the following:

  • Architecture redesign
  • API changes
  • Changes in CLI flags
  • New/Updated indexes

Misc

  • PR title will be clear as part of the changelog
  • PR is against the correct branch
  • PR is labelled appropriately
  • PR is linked to an issue

GetElastech added 5 commits July 6, 2022 10:32
Author: Miklos Szegedi
Author: Miklos Szegedi
Author: Miklos Szegedi
Author: Miklos Szegedi
Author: Miklos Szegedi
@miklosszegedi

Copy link
Copy Markdown
Author

The goal of this pull request is to suggest some changes to make DPS CI finish successfully.

Comment thread service/tracker/execution.go Outdated
Author: Miklos Szegedi Reviewer: Peter Argue
@miklosszegedi

Copy link
Copy Markdown
Author

The issue remaining is an undefined crypto.NewBLSKMAC

@miklosszegedi miklosszegedi marked this pull request as draft July 6, 2022 19:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants