Skip to content
This repository has been archived by the owner on Feb 21, 2024. It is now read-only.

refactor: remove generator script #306

Merged
merged 9 commits into from
Nov 16, 2023
Merged

refactor: remove generator script #306

merged 9 commits into from
Nov 16, 2023

Conversation

alestiago
Copy link
Contributor

@alestiago alestiago commented Nov 2, 2023 •
edited
Loading

Description

Resolves #304

Changes:

  • Removes generator script
  • Removes e2e test of generator script
  • Adds workflow to test brick generation

Type of Change

  • ✨ New feature (non-breaking change which adds functionality)
  • 🛠️ Bug fix (non-breaking change which fixes an issue)
  • ❌ Breaking change (fix or feature that would cause existing functionality to change)
  • 🧹 Code refactor
  • ✅ Build configuration change
  • 📝 Documentation
  • 🗑️ Chore

@alestiago alestiago added the refactor Refactor an existing implementation label Nov 2, 2023
@alestiago alestiago self-assigned this Nov 2, 2023
@alestiago alestiago mentioned this pull request Nov 6, 2023
Merged
7 tasks
@alestiago alestiago marked this pull request as ready for review November 15, 2023 11:29
@alestiago alestiago requested a review from a team as a code owner November 15, 2023 11:29
@alestiago alestiago merged commit 24c5f9a into main Nov 16, 2023
3 checks passed
@alestiago alestiago deleted the alestiago/remove-generator branch November 16, 2023 16:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@renancaraujo renancaraujo renancaraujo approved these changes

@wolfenrain wolfenrain wolfenrain approved these changes

Assignees

@alestiago alestiago

Labels
refactor Refactor an existing implementation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

refactor: remove generator script
3 participants
@alestiago @renancaraujo @wolfenrain