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

Fixes to the Stratus architecture file and the faust2stratus script #1022

Closed
wants to merge 3 commits into from

Conversation

bassmanitram
Copy link

The architecture file has been optimized to remove a lot of unnecessary pointer redirections
The script has been fixed to be much tighter about output file naming and location

@sletz
Copy link
Member

sletz commented May 24, 2024

This branch cannot be rebased due to conflicts ==> I always to "rebase merging" to keep the linear tree.

You still have tab/space issues in your code and that cause merge conflicts, so:

  • can you use 1 Tab = 4 spaces formatting model in your files?
  • and rewrote you change to allows a proper rebase merging ?

Thanks !

@bassmanitram
Copy link
Author

Certainly! Can you just clarify - you want tabs in the code, or spaces in the code?

@bassmanitram
Copy link
Author

bassmanitram commented May 24, 2024

It's probably best to close this PR and do it again, eh :)

@sletz
Copy link
Member

sletz commented May 24, 2024

only spaces in the code, thanks !

@sletz
Copy link
Member

sletz commented May 24, 2024

Yes, a new PR is event better, thanks.

@bassmanitram bassmanitram closed this by deleting the head repository May 24, 2024
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.

None yet

2 participants