rdenadai/rdenadai.com.br
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat: New blog posts #1

Merged
rdenadai merged 1 commit from rdenadai/pipe-operator-python into main 2025-09-05 03:34:05 +00:00
Conversation 1 Commits 1 Files changed 8 +895
rdenadai commented 2025-09-05 03:33:57 +00:00 (Migrated from github.com)
Copy link
No description provided.
copilot-pull-request-reviewer[bot] (Migrated from github.com) reviewed 2025-09-05 03:35:23 +00:00
copilot-pull-request-reviewer[bot] (Migrated from github.com) left a comment
Copy link

Pull Request Overview

A new blog post addition focused on implementing a pipe operator in Python, along with metadata updates to support the new content.

  • Added Portuguese and English versions of a new essay about mimicking pipeline operators in Python
  • Added a book quote essay about software design philosophy
  • Updated essays metadata configuration to include the new entries

Reviewed Changes

Copilot reviewed 4 out of 8 changed files in this pull request and generated 7 comments.

File Description
static/pages/essays/28/README.pt-BR.md Portuguese version of the pipe operator essay with technical implementation examples
static/pages/essays/28/README.md English version of the pipe operator essay with code examples and explanations
static/pages/essays/27/README.md Short essay featuring a quote from "A Philosophy of Software Design" book
static/pages/essays.json Metadata entries for the new essays including titles, descriptions, and tags

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

## Pull Request Overview A new blog post addition focused on implementing a pipe operator in Python, along with metadata updates to support the new content. - Added Portuguese and English versions of a new essay about mimicking pipeline operators in Python - Added a book quote essay about software design philosophy - Updated essays metadata configuration to include the new entries ### Reviewed Changes Copilot reviewed 4 out of 8 changed files in this pull request and generated 7 comments. | File | Description | | ---- | ----------- | | static/pages/essays/28/README.pt-BR.md | Portuguese version of the pipe operator essay with technical implementation examples | | static/pages/essays/28/README.md | English version of the pipe operator essay with code examples and explanations | | static/pages/essays/27/README.md | Short essay featuring a quote from "A Philosophy of Software Design" book | | static/pages/essays.json | Metadata entries for the new essays including titles, descriptions, and tags | --- <sub>**Tip:** Customize your code reviews with copilot-instructions.md. <a href="/rdenadai/rdenadai.com.br/new/main/.github?filename=copilot-instructions.md" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Create the file</a> or <a href="https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">learn how to get started</a>.</sub>
static/pages/essays.json
@ -350,0 +388,4 @@
"alternate": "aspects-and-usage-of-llms-in-learning-and-school",
"path": "static/pages/essays/29",
"title": "The Aspects and Usage of LLMs in Learning and School",
"description": ".",
copilot-pull-request-reviewer[bot] (Migrated from github.com) commented 2025-09-05 03:35:23 +00:00
Copy link

The description for essay ID 29 is just a period, which appears to be incomplete or a placeholder. Consider adding a proper description or removing this entry if it's not ready for publication.

    "description": "This essay explores the various aspects and practical uses of Large Language Models (LLMs) in educational settings, discussing their benefits, challenges, and implications for learning and teaching.",
The description for essay ID 29 is just a period, which appears to be incomplete or a placeholder. Consider adding a proper description or removing this entry if it's not ready for publication. ```suggestion "description": "This essay explores the various aspects and practical uses of Large Language Models (LLMs) in educational settings, discussing their benefits, challenges, and implications for learning and teaching.", ```
static/pages/essays/28/README.md
@ -0,0 +43,4 @@
def upper(s: str) -> str:
return s.upper()
def split(s: str, char: str = " ") -> str:
copilot-pull-request-reviewer[bot] (Migrated from github.com) commented 2025-09-05 03:35:22 +00:00
Copy link

The return type annotation should be List[str] instead of str since the split() method returns a list of strings.

from typing import List

def upper(s: str) -> str:
    return s.upper()

def split(s: str, char: str = " ") -> List[str]:
The return type annotation should be `List[str]` instead of `str` since the `split()` method returns a list of strings. ```suggestion from typing import List def upper(s: str) -> str: return s.upper() def split(s: str, char: str = " ") -> List[str]: ```
static/pages/essays/28/README.md
@ -0,0 +215,4 @@
return self._func(other)
if isinstance(other, partial):
return self._func(other())
copilot-pull-request-reviewer[bot] (Migrated from github.com) commented 2025-09-05 03:35:22 +00:00
Copy link

The Portuguese version has a # type: ignore comment on this line that's missing in the English version. Consider adding it for consistency if the type checker requires it, or address the underlying type issue in both versions.

            return self._func(other())  # type: ignore
The Portuguese version has a `# type: ignore` comment on this line that's missing in the English version. Consider adding it for consistency if the type checker requires it, or address the underlying type issue in both versions. ```suggestion return self._func(other()) # type: ignore ```
static/pages/essays/28/README.md
@ -0,0 +392,4 @@
print(x % 2 == 0)
yield x % 2 == 0
def multiply_by_10(x: int) -> int:
copilot-pull-request-reviewer[bot] (Migrated from github.com) commented 2025-09-05 03:35:22 +00:00
Copy link

The return type annotation should be Generator[int, None, None] instead of int since this function yields values instead of returning them.

from typing import Generator

@pipe
def generator_func() -> Generator[int, None, None]:
    for i in range(6):
        print(">>>", i)
        yield i

def filter_divisible_by_3(x: int) -> Generator[bool, None, None]:
    print(x % 2 == 0)
    yield x % 2 == 0

def multiply_by_10(x: int) -> Generator[int, None, None]:
The return type annotation should be `Generator[int, None, None]` instead of `int` since this function yields values instead of returning them. ```suggestion from typing import Generator @pipe def generator_func() -> Generator[int, None, None]: for i in range(6): print(">>>", i) yield i def filter_divisible_by_3(x: int) -> Generator[bool, None, None]: print(x % 2 == 0) yield x % 2 == 0 def multiply_by_10(x: int) -> Generator[int, None, None]: ```
static/pages/essays/28/README.pt-BR.md
@ -0,0 +43,4 @@
def upper(s: str) -> str:
return s.upper()
def split(s: str, char: str = " ") -> str:
copilot-pull-request-reviewer[bot] (Migrated from github.com) commented 2025-09-05 03:35:21 +00:00
Copy link

The return type annotation should be List[str] instead of str since the split() method returns a list of strings.

from typing import List

def upper(s: str) -> str:
    return s.upper()

def split(s: str, char: str = " ") -> List[str]:
The return type annotation should be `List[str]` instead of `str` since the `split()` method returns a list of strings. ```suggestion from typing import List def upper(s: str) -> str: return s.upper() def split(s: str, char: str = " ") -> List[str]: ```
static/pages/essays/28/README.pt-BR.md
@ -0,0 +215,4 @@
return self._func(other)
if isinstance(other, partial):
return self._func(other()) # type: ignore
copilot-pull-request-reviewer[bot] (Migrated from github.com) commented 2025-09-05 03:35:22 +00:00
Copy link

The # type: ignore comment suggests type safety issues. Consider using proper type annotations or type guards to resolve the underlying type checking problem rather than suppressing it.

The `# type: ignore` comment suggests type safety issues. Consider using proper type annotations or type guards to resolve the underlying type checking problem rather than suppressing it.
static/pages/essays/28/README.pt-BR.md
@ -0,0 +385,4 @@
def multiply_by_10(x: int) -> int:
print(x * 10)
yield x * 10
copilot-pull-request-reviewer[bot] (Migrated from github.com) commented 2025-09-05 03:35:22 +00:00
Copy link

The return type annotation should be Generator[int, None, None] instead of int since this function yields values instead of returning them.

The return type annotation should be `Generator[int, None, None]` instead of `int` since this function yields values instead of returning them.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something isn't working

  
documentation

Improvements or additions to documentation

  
duplicate

This issue or pull request already exists

  
enhancement

New feature or request

  
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
invalid

This doesn't seem right

  
question

Further information is requested

  
wontfix

This will not be worked on

No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
rdenadai/rdenadai.com.br!1
No description provided.