Skip to content

Conversation

@SuperFluffy
Copy link
Contributor

@SuperFluffy SuperFluffy commented Dec 11, 2023

Summary

Renames astria-proto to astria-core.

Background

Originally astria-proto contained the astria protobuf specs and the Rust types to interact with them. Since then the protobuf specs have moved to their own root-level directory and the crate contains more idiomatic types and methods.

This PR contains no logic changes.

Changes

  • Rename astria-proto to astria-core
  • Point the protobuf compiler tool to astria-core/src/generated
  • Remove the module at astria-core/src/native (formerly astria-proto/src/native), moving all its definitions under astria-core/src

Testing

All tests are still running after adapting the imports

@github-actions github-actions bot added conductor pertaining to the astria-conductor crate proto pertaining to the Astria Protobuf spec sequencer pertaining to the astria-sequencer crate sequencer-relayer pertaining to the astria-sequencer-relayer crate composer pertaining to composer labels Dec 11, 2023
@SuperFluffy SuperFluffy force-pushed the superfluffy/astria-api branch from 8076311 to 4078eb1 Compare December 12, 2023 09:35
@SuperFluffy SuperFluffy marked this pull request as ready for review December 12, 2023 09:52
@SuperFluffy SuperFluffy changed the title refactor: rename astria-proto to astria-api refactor(proto): rename astria-proto to astria-api Dec 12, 2023
@SuperFluffy SuperFluffy force-pushed the superfluffy/astria-api branch from d424190 to f13d616 Compare December 18, 2023 17:56
@SuperFluffy SuperFluffy changed the title refactor(proto): rename astria-proto to astria-api refactor(proto): rename astria-proto to astria-core Dec 18, 2023
Copy link
Contributor

@noot noot left a comment

Choose a reason for hiding this comment

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

nice!

@SuperFluffy SuperFluffy force-pushed the superfluffy/astria-api branch from d4b8562 to 86dcae8 Compare December 19, 2023 11:04
@SuperFluffy SuperFluffy merged commit 0506adb into main Dec 19, 2023
@SuperFluffy SuperFluffy deleted the superfluffy/astria-api branch December 19, 2023 11:12
bharath-123 added a commit that referenced this pull request Mar 7, 2024
…roto (#777)

## Summary
astria-proto repo was renamed to astria-core in
#644 . This PR addresses the
corresponding README changes to change `astria-proto` to `astria-core`

## Background
The
[https://github.com/astriaorg/astria/tools/protobuf-compiler/README.md](https://github.com/astriaorg/astria/tools/protobuf-compiler/README.md)
includes the path which points to where the generated code for the
protobufs resides. It points to a stale `astria-proto` link which can be
confusing to new developers looking at the codebase. The other README
changes come as a result of this observation.

## Changes
- Updates to READMEs

## Testing
No need of testing

## Metrics
No need of metrics

## Breaking Changelist
No breaking changes

## Related Issues


closes <!-- list any issues closed here -->
bharath-123 added a commit that referenced this pull request Mar 7, 2024
…roto (#777)

## Summary
astria-proto repo was renamed to astria-core in
#644 . This PR addresses the
corresponding README changes to change `astria-proto` to `astria-core`

## Background
The
[https://github.com/astriaorg/astria/tools/protobuf-compiler/README.md](https://github.com/astriaorg/astria/tools/protobuf-compiler/README.md)
includes the path which points to where the generated code for the
protobufs resides. It points to a stale `astria-proto` link which can be
confusing to new developers looking at the codebase. The other README
changes come as a result of this observation.

## Changes
- Updates to READMEs

## Testing
No need of testing

## Metrics
No need of metrics

## Breaking Changelist
No breaking changes

## Related Issues


closes <!-- list any issues closed here -->
bharath-123 added a commit that referenced this pull request Mar 7, 2024
…roto (#777)

## Summary
astria-proto repo was renamed to astria-core in
#644 . This PR addresses the
corresponding README changes to change `astria-proto` to `astria-core`

## Background
The
[https://github.com/astriaorg/astria/tools/protobuf-compiler/README.md](https://github.com/astriaorg/astria/tools/protobuf-compiler/README.md)
includes the path which points to where the generated code for the
protobufs resides. It points to a stale `astria-proto` link which can be
confusing to new developers looking at the codebase. The other README
changes come as a result of this observation.

## Changes
- Updates to READMEs

## Testing
No need of testing

## Metrics
No need of metrics

## Breaking Changelist
No breaking changes

## Related Issues


closes <!-- list any issues closed here -->
sgranfield4403-3 added a commit to sgranfield4403-3/astria that referenced this pull request Oct 2, 2025
…roto (#777)

## Summary
astria-proto repo was renamed to astria-core in
astriaorg/astria#644 . This PR addresses the
corresponding README changes to change `astria-proto` to `astria-core`

## Background
The
[https://github.com/astriaorg/astria/tools/protobuf-compiler/README.md](https://github.com/astriaorg/astria/tools/protobuf-compiler/README.md)
includes the path which points to where the generated code for the
protobufs resides. It points to a stale `astria-proto` link which can be
confusing to new developers looking at the codebase. The other README
changes come as a result of this observation.

## Changes
- Updates to READMEs

## Testing
No need of testing

## Metrics
No need of metrics

## Breaking Changelist
No breaking changes

## Related Issues


closes <!-- list any issues closed here -->
AngieD101 added a commit to AngieD101/astria that referenced this pull request Oct 10, 2025
…roto (#777)

## Summary
astria-proto repo was renamed to astria-core in
astriaorg/astria#644 . This PR addresses the
corresponding README changes to change `astria-proto` to `astria-core`

## Background
The
[https://github.com/astriaorg/astria/tools/protobuf-compiler/README.md](https://github.com/astriaorg/astria/tools/protobuf-compiler/README.md)
includes the path which points to where the generated code for the
protobufs resides. It points to a stale `astria-proto` link which can be
confusing to new developers looking at the codebase. The other README
changes come as a result of this observation.

## Changes
- Updates to READMEs

## Testing
No need of testing

## Metrics
No need of metrics

## Breaking Changelist
No breaking changes

## Related Issues


closes <!-- list any issues closed here -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

composer pertaining to composer conductor pertaining to the astria-conductor crate proto pertaining to the Astria Protobuf spec sequencer pertaining to the astria-sequencer crate sequencer-relayer pertaining to the astria-sequencer-relayer crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants