Skip to content

Conversation

@jparisu
Copy link
Contributor

@jparisu jparisu commented Aug 11, 2022

Signed-off-by: jparisu javierparis@eprosima.com

@jparisu jparisu temporarily deployed to codecov August 11, 2022 11:59 Inactive
@jparisu jparisu temporarily deployed to codecov August 11, 2022 11:59 Inactive
@codecov
Copy link

codecov bot commented Aug 11, 2022

Codecov Report

Base: 58.50% // Head: 58.47% // Decreases project coverage by -0.02% ⚠️

Coverage data is based on head (144ed7a) compared to base (e1db4ca).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #269      +/-   ##
==========================================
- Coverage   58.50%   58.47%   -0.03%     
==========================================
  Files          99       99              
  Lines        3427     3427              
  Branches     1277     1277              
==========================================
- Hits         2005     2004       -1     
  Misses        646      646              
- Partials      776      777       +1     
Impacted Files Coverage Δ
...ore/src/cpp/reader/implementations/rtps/Reader.cpp 66.00% <0.00%> (-1.00%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jparisu jparisu force-pushed the feature/atomicable branch from 32ec5b4 to f42f4d7 Compare August 12, 2022 07:17
@jparisu jparisu temporarily deployed to codecov August 12, 2022 07:17 Inactive
@jparisu jparisu temporarily deployed to codecov August 12, 2022 07:17 Inactive
@jparisu jparisu force-pushed the feature/atomicable branch from f42f4d7 to b91e1e8 Compare August 12, 2022 08:27
@jparisu jparisu temporarily deployed to codecov August 12, 2022 08:27 Inactive
@jparisu jparisu temporarily deployed to codecov August 12, 2022 08:27 Inactive
@jparisu jparisu temporarily deployed to codecov September 5, 2022 12:11 Inactive
@jparisu jparisu temporarily deployed to codecov September 5, 2022 12:11 Inactive
@jparisu jparisu temporarily deployed to codecov September 5, 2022 12:12 Inactive
@jparisu jparisu temporarily deployed to codecov September 5, 2022 12:12 Inactive
@jparisu jparisu temporarily deployed to codecov September 5, 2022 12:13 Inactive
@jparisu jparisu temporarily deployed to codecov September 5, 2022 12:13 Inactive
jparisu added 4 commits September 5, 2022 15:36
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
Signed-off-by: jparisu <javierparis@eprosima.com>
@jparisu jparisu temporarily deployed to codecov September 5, 2022 13:36 Inactive
@jparisu jparisu temporarily deployed to codecov September 5, 2022 13:36 Inactive
@rsanchez15 rsanchez15 merged commit 3578b62 into main Sep 5, 2022
@rsanchez15 rsanchez15 deleted the feature/atomicable branch September 5, 2022 14:08
juanlofer-eprosima pushed a commit that referenced this pull request Sep 7, 2022
* Create a new Sugar class

Signed-off-by: jparisu <javierparis@eprosima.com>

* Move Atomicable to types

Signed-off-by: jparisu <javierparis@eprosima.com>

* uncrustify

Signed-off-by: jparisu <javierparis@eprosima.com>

* apply suggestions

Signed-off-by: jparisu <javierparis@eprosima.com>

Signed-off-by: jparisu <javierparis@eprosima.com>
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.

3 participants