Skip to content

Feature/hacktoberfest#111

Open
profrod-ai wants to merge 3 commits intomainfrom
feature/hacktoberfest
Open

Feature/hacktoberfest#111
profrod-ai wants to merge 3 commits intomainfrom
feature/hacktoberfest

Conversation

@profrod-ai
Copy link
Copy Markdown
Contributor

Adding changes to the readme and to .github to enhance contributor participation during hacktoberfest

@char0n char0n self-requested a review September 12, 2025 08:47
@char0n char0n assigned char0n and profrod-ai and unassigned char0n Sep 12, 2025
@char0n char0n added documentation Improvements or additions to documentation enhancement New feature or request labels Sep 12, 2025
Copy link
Copy Markdown
Member

@char0n char0n left a comment

Choose a reason for hiding this comment

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

Hi @rodjentic,

Thank you for the PR! Left some inline core reviews comments.

# Jupyter Notebooks
# -------------------------
.ipynb_checkpoints/
*.ipynb
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why do we want to ignore all notebooks?

# -------------------------
# Docker
# -------------------------
.dockerignore
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why do we want to ignore .dockerignore file ?

# -------------------------
# Project Specific
# -------------------------
# Arazzo Generator
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Shouldn't this go for project specific .gitignore?

generator/_transient-files/
generator/extracted_json.txt

# Arazzo Runner
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Shouldn't this go for project specific .gitignore?

# LLM Interaction logs
*_llm_interaction/

# Generated specs and workflows
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Shouldn't this go for project specific .gitignore?

## Related Projects

### Official Arazzo Resources
- [Arazzo Specification](https://github.com/OAI/Arazzo-Specification) - Official spec repository
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

OpenAPI repos are no longer authoritative source of truth. Can we use authoritative URL for the Arazzo spec here? https://spec.openapis.org/arazzo/latest.html

- Advanced AI integrations
- Workflow marketplace

[**View detailed roadmap →**](https://github.com/jentic/arazzo-engine/projects)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Links to nowhere

- **Arazzo Generator**: 🚧 Beta (v0.2+) - Active development
- **Specification Compliance**: ✅ Arazzo 1.0.1 fully supported

### 📋 Roadmap
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Are we sure about this roadmap? It's seems LLM generated and not aligned with reality.

### 🏆 Recognition

All contributors are recognized in our:
- [Contributors list](CONTRIBUTORS.md)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We don't have contributors list and we probably don't want to maintain one manually.

### 💬 Join the Conversation

- **Discord**: [Join our community](https://discord.gg/TdbWXZsUSm) for real-time discussions
- **GitHub Discussions**: Share ideas and ask questions
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe a link to discussions when we have org wide one?


### 🚀 Current Status

- **Arazzo Runner**: ✅ Stable (v1.0+) - Production ready
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
- **Arazzo Runner**: ✅ Stable (v1.0+) - Production ready
- **Arazzo Runner**: ✅ Stable (v0.9+) - Active development


- **Arazzo Runner**: ✅ Stable (v1.0+) - Production ready
- **Arazzo Generator**: 🚧 Beta (v0.2+) - Active development
- **Specification Compliance**: ✅ Arazzo 1.0.1 fully supported
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Arazzo 1.0.1 fully supported - this is very strong claim and might set expectation ladder too high. We're aiming to be Arazzo compliant at this stage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants