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

File parser #744

Merged
merged 5 commits into from
May 21, 2024
Merged

File parser #744

merged 5 commits into from
May 21, 2024

Conversation

melkiyasser
Copy link
Contributor

file parser , read a file parse then its content with microdown parser and add the file to the properties of the root element .

@@ -0,0 +1,30 @@
Class {
#name : 'FileParser',
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you rename your class FromFileTest?
tx

Microdown >> parseFile: aFile [

|root|
root := MicrodownParser parse:(aFile contents) .
Copy link
Contributor

Choose a reason for hiding this comment

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

parse: (aFile contents).

No extra space

^ self propertyAt: #file .
]

{ #category : 'visiting' }
Copy link
Contributor

Choose a reason for hiding this comment

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

Not in visiting

@@ -32,6 +32,16 @@ MicRootBlock >> canConsumeLine: line [
^ true
]

{ #category : 'visiting' }
MicRootBlock >> fromFile [ " retieve "
^ self propertyAt: #file .
Copy link
Contributor

Choose a reason for hiding this comment

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

the last period is not needed.

@Ducasse
Copy link
Contributor

Ducasse commented May 21, 2024

I could not understand why the ParentChild code is in this PR.

@Ducasse
Copy link
Contributor

Ducasse commented May 21, 2024

I will merge this PR but it was messy.
Please have a look at my comments.

@Ducasse Ducasse merged commit 35858cf into pillar-markup:dev May 21, 2024
2 checks passed
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.

2 participants