Possibility to exclude specific object from handling #70

LastSprint · 2021-10-26T11:12:49Z

#56 done!

Added AST tree shaking just after Yaml parsing. Seems works :D

Sources/Common/PathNormalizer.swift

Sources/Pipelines/CLI/Utils/Urls.swift

Co-authored-by: Margarita Volodina <volodina@surfstudio.ru>

Sources/Pipelines/CLI/Utils/Urls.swift

dmitryd20

Feature works fine, except for some cases

Sources/ASTTree/ASTNodeExcluder.swift

…https://github.com/surfstudio/SurfGen into possibility-to-exclude-specific-object-from-handling

.surfgen.generator.yaml

Sources/Pipelines/CLI/Utils/Urls.swift

Co-authored-by: Dmitry Demianov <dmitryd20@gmail.com>

.surfgen.generator.yaml

.surfgen.linter.yaml

README.md

Sources/Pipelines/BuldGASTTree/BuildGASTTreeEntryPoint.swift

Sources/Pipelines/CLI/CreateConfig/ConfigExample.swift

margarita-v · 2021-10-29T09:57:24Z

надо бы выполнить одно из действий

обновить бинарник в репо согласно новой логике
автоматизировать обновление бинарника при мерже в репо
удалить бинарник, чтобы пользователи сами выполняли make executable

в идеале сделать бы 2 опцию, мб отдельно, провайдить бинарник для мака и линукса

Package.resolved

Co-authored-by: Margarita Volodina <volodina@surfstudio.ru>

LastSprint · 2021-11-01T07:44:17Z

надо бы выполнить одно из действий

обновить бинарник в репо согласно новой логике

автоматизировать обновление бинарника при мерже в репо

удалить бинарник, чтобы пользователи сами выполняли make executable

в идеале сделать бы 2 опцию, мб отдельно, провайдить бинарник для мака и линукса

Не в этом PR. Давай ишую

Co-authored-by: Margarita Volodina <volodina@surfstudio.ru>

LastSprint added 4 commits October 21, 2021 21:27

add AST builder and initiated tests

62cc328

add tests to ast collector

d43e835

insert AST builder in pipeline and check that all tests have passed

7f96cd3

add ability to cut some AST nodes and add docs and tests

9ece5fa

LastSprint requested review from chausovSurfStudio, MbIXjkee, margarita-v and dmitryd20 October 26, 2021 11:12

LastSprint self-assigned this Oct 26, 2021

LastSprint added 2 commits October 26, 2021 14:14

fix readme

b3c43de

small refactoring

db97f71

margarita-v reviewed Oct 26, 2021

View reviewed changes

Sources/Common/PathNormalizer.swift Outdated Show resolved Hide resolved

margarita-v reviewed Oct 26, 2021

View reviewed changes

Sources/Pipelines/CLI/Utils/Urls.swift Outdated Show resolved Hide resolved

Alexander Kravchenkov and others added 2 commits October 27, 2021 07:47

Update Sources/Pipelines/CLI/Utils/Urls.swift

01a9940

Co-authored-by: Margarita Volodina <volodina@surfstudio.ru>

Update PathNormalizer.swift

af1212c

dmitryd20 reviewed Oct 27, 2021

View reviewed changes

Sources/Pipelines/CLI/Utils/Urls.swift Show resolved Hide resolved

dmitryd20 requested changes Oct 27, 2021

View reviewed changes

Sources/ASTTree/ASTNodeExcluder.swift Show resolved Hide resolved

LastSprint added 3 commits October 27, 2021 15:36

fixed issue with absolute path in generator

342ea9d

Merge branch 'possibility-to-exclude-specific-object-from-handling' of …

b8f4756

…https://github.com/surfstudio/SurfGen into possibility-to-exclude-specific-object-from-handling

add replacing in paths

277de51

LastSprint requested review from margarita-v and dmitryd20 October 28, 2021 07:29

dmitryd20 requested changes Oct 28, 2021

View reviewed changes

.surfgen.generator.yaml Outdated Show resolved Hide resolved

Sources/Pipelines/CLI/Utils/Urls.swift Outdated Show resolved Hide resolved

fix pr issues

3ff349b

LastSprint requested a review from dmitryd20 October 28, 2021 09:47

dmitryd20 reviewed Oct 28, 2021

View reviewed changes

Sources/Pipelines/CLI/Utils/Urls.swift Outdated Show resolved Hide resolved

LastSprint and others added 2 commits October 28, 2021 12:57

fix test api speck

e7ac048

Update Sources/Pipelines/CLI/Utils/Urls.swift

29b15a9

Co-authored-by: Dmitry Demianov <dmitryd20@gmail.com>

LastSprint requested a review from dmitryd20 October 28, 2021 10:18

dmitryd20 approved these changes Oct 28, 2021

View reviewed changes

margarita-v reviewed Oct 29, 2021

View reviewed changes

Package.resolved Show resolved Hide resolved

Alexander Kravchenkov and others added 6 commits November 1, 2021 10:42

Update .surfgen.generator.yaml

392b020

Co-authored-by: Margarita Volodina <volodina@surfstudio.ru>

Update .surfgen.linter.yaml

c38d822

Co-authored-by: Margarita Volodina <volodina@surfstudio.ru>

Update Sources/Pipelines/CLI/CreateConfig/ConfigExample.swift

b505e9a

Co-authored-by: Margarita Volodina <volodina@surfstudio.ru>

Update Sources/Pipelines/CLI/CreateConfig/ConfigExample.swift

a30a18d

Co-authored-by: Margarita Volodina <volodina@surfstudio.ru>

Update .surfgen.generator.yaml

5724bbf

Co-authored-by: Margarita Volodina <volodina@surfstudio.ru>

Update Sources/Pipelines/CLI/CreateConfig/ConfigExample.swift

28202f5

Co-authored-by: Margarita Volodina <volodina@surfstudio.ru>

Alexander Kravchenkov and others added 2 commits November 1, 2021 10:45

Update README.md

a65b3a7

Co-authored-by: Margarita Volodina <volodina@surfstudio.ru>

remove unnecessery todo

7cda6b4

LastSprint merged commit c9a1af1 into master Nov 1, 2021

LastSprint deleted the possibility-to-exclude-specific-object-from-handling branch November 1, 2021 07:54

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Possibility to exclude specific object from handling #70

Possibility to exclude specific object from handling #70

LastSprint commented Oct 26, 2021 •

edited

Loading

dmitryd20 left a comment

margarita-v commented Oct 29, 2021 •

edited

Loading

LastSprint commented Nov 1, 2021

Possibility to exclude specific object from handling #70

Possibility to exclude specific object from handling #70

Conversation

LastSprint commented Oct 26, 2021 • edited Loading

dmitryd20 left a comment

Choose a reason for hiding this comment

margarita-v commented Oct 29, 2021 • edited Loading

LastSprint commented Nov 1, 2021

LastSprint commented Oct 26, 2021 •

edited

Loading

margarita-v commented Oct 29, 2021 •

edited

Loading